Transaction state is preserved across retries

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Unresolved
    • Priority: Unknown
    • None
    • Component/s: Backpressure, Transactions
    • None
    • Needed - No Spec Changes
    • Hide

      Summary of necessary driver changes

      Commits for syncing spec/prose tests
      (and/or refer to an existing language POC if needed)
      * https://github.com/mongodb/specifications/commit/7789704effa04ccde11d7e383c23203e2dd0c509

      Context for other referenced/linked tickets

      •  Extended the command assertions in a single test (retry succeeds if backpressure labels are added to the first operation in a transaction) to verify that startTransaction: true is present on both insert (the initial attempt and retry).
      Show
      Summary of necessary driver changes  This is an addition to DRIVERS-3239 Commits for syncing spec/prose tests (and/or refer to an existing language POC if needed) *  https://github.com/mongodb/specifications/commit/7789704effa04ccde11d7e383c23203e2dd0c509 Context for other referenced/linked tickets  Extended the command assertions in a single test (retry succeeds if backpressure labels are added to the first operation in a transaction) to verify that startTransaction: true is present on both insert (the initial attempt and retry).
    • $i18n.getText("admin.common.words.hide")
      Key Status/Resolution FixVersion
      CDRIVER-6256 Needs Triage
      CXX-3428 Needs Triage
      CSHARP-5906 Needs Triage
      GODRIVER-3830 Needs Triage
      JAVA-6117 Needs Triage
      NODE-7463 Needs Verification
      PYTHON-5741 Done
      PHPLIB-1801 Needs Triage
      RUBY-3786 Needs Triage
      RUST-2373 Needs Triage
      $i18n.getText("admin.common.words.show")
      #scriptField, #scriptField *{ border: 1px solid black; } #scriptField{ border-collapse: collapse; } #scriptField td { text-align: center; /* Center-align text in table cells */ } #scriptField td.key { text-align: left; /* Left-align text in the Key column */ } #scriptField a { text-decoration: none; /* Remove underlines from links */ border: none; /* Remove border from links */ } /* Add green background color to cells with FixVersion */ #scriptField td.hasFixVersion { background-color: #00FF00; /* Green color code */ } #scriptField td.willNotDo { background-color: #FF0000; /* Red color code */ } /* Center-align the first row headers */ #scriptField th { text-align: center; } Key Status/Resolution FixVersion CDRIVER-6256 Needs Triage CXX-3428 Needs Triage CSHARP-5906 Needs Triage GODRIVER-3830 Needs Triage JAVA-6117 Needs Triage NODE-7463 Needs Verification PYTHON-5741 Done PHPLIB-1801 Needs Triage RUBY-3786 Needs Triage RUST-2373 Needs Triage

      Summary

      Existing test introduced in DRIVERS-3239 "retry succeeds if backpressure labels are added to the first operation in a transaction" only verifies commands sequence, but doesn't verify the content of the command, which is crucial, as the first attempt doesn't create transaction on server which means the second command must preserve startTransaction: true._

      Motivation

      Who is the affected end user?

      Who are the stakeholders?

      How does this affect the end user?

      Are they blocked? Are they annoyed? Are they confused?

      How likely is it that this problem or use case will occur?

      Main path? Edge case?

      If the problem does occur, what are the consequences and how severe are they?

      Minor annoyance at a log message? Performance concern? Outage/unavailability? Failover can't complete?

      Is this issue urgent?

      Does this ticket have a required timeline? What is it?

      Is this ticket required by a downstream team?

      Needed by e.g. Atlas, Shell, Compass?

      Is this ticket only for tests?

      Does this ticket have any functional impact, or is it just test improvements?

      Acceptance Criteria

      Add command fields to compare for the test.

            Assignee:
            Sergey Zelenov
            Reporter:
            Sergey Zelenov
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: