Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-6190 Using Promise.all inside transactions errors with "Cannot pin multiple connections to the same session" on serverless instance
  3. NODE-6192

[v5.x BACKPORT] Add inline documentation to withTransaction and startTransaction to clarify that Promise.all etc should not be used during transactions

    • Type: Icon: Sub-task Sub-task
    • Resolution: Fixed
    • Priority: Icon: Unknown Unknown
    • 6.8.0
    • Affects Version/s: None
    • Component/s: None
    • None
    • 0
    • Not Needed
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?

      Backport NODE-6191 to v5.x.

      Parent Ticket Description:

      Use Case

      As an... Application developer
      I want... documentation that explains details undefined behaviour
      So that... __ I can avoid known undefined behaviours

      User Impact

      • Unknown, but a number of external and internal users have expressed confusion around this.

      Dependencies

      • None

      Unknowns

      • None

      Acceptance Criteria

      Implementation Requirements

      • Add inline documentation to Session.withTransaction and Session.startTransaction that warn against running operations in parallel in a transaction

      Testing Requirements

      • None

      Documentation Requirements

      • None

      Follow Up Requirements

      • None

            Assignee:
            warren.james@mongodb.com Warren James
            Reporter:
            dbeng-pm-bot PM Bot
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: