Audit call sites of commitTransaction_forTesting() and abortTransaction_forTesting() to make sure they do not ignore server response

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Fixed
    • Priority: Major - P3
    • 4.1.14, 4.2.0-rc1
    • Affects Version/s: None
    • Component/s: Replication
    • None
    • Fully Compatible
    • v4.2
    • Repl 2019-05-06, Repl 2019-06-03, Repl 2019-06-17
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      As a part of SERVER-36728, we replaced the abortTransaction() helper with abortTransaction_forTesting(), which does not ignore server errors. However, in a majority of our tests, we do not use the response.

      We should either assert that the abort succeeded or that the command failed with NoSuchTransaction to account for transactions implicitly aborted by the server.

      We should also make sure we aren't using commitTransaction() or abortTransaction() in our tests unintentionally.

            Assignee:
            Lingzhi Deng
            Reporter:
            Pavithra Vetriselvan
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

              Created:
              Updated:
              Resolved: