-
Type: Improvement
-
Resolution: Fixed
-
Priority: Major - P3
-
Affects Version/s: None
-
Component/s: Replication
-
None
-
Fully Compatible
-
v4.2
-
Repl 2019-05-06, Repl 2019-06-03, Repl 2019-06-17
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.
- depends on
-
SERVER-41127 TransactionRouter should not assert that a participant's readOnly flag is set if the transaction is being aborted
- Closed