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/1125200e4a6161d87cb5090860597eb8e8e90bf1

      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/1125200e4a6161d87cb5090860597eb8e8e90bf1 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 Done 2.3.0
      CXX-3428 Backlog
      CSHARP-5906 Done
      GODRIVER-3830 Backlog
      JAVA-6117 In Progress
      NODE-7463 Fixed 7.2.0
      PYTHON-5741 Done
      PHPLIB-1801 Backlog
      RUBY-3786 Fixed 2.24.0
      RUST-2373 Fixed 3.6.0
      $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 Done 2.3.0 CXX-3428 Backlog CSHARP-5906 Done GODRIVER-3830 Backlog JAVA-6117 In Progress NODE-7463 Fixed 7.2.0 PYTHON-5741 Done PHPLIB-1801 Backlog RUBY-3786 Fixed 2.24.0 RUST-2373 Fixed 3.6.0

      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: