[SERVER-51598] Add new test suites that test transaction expiration logic Created: 14/Oct/20 Updated: 29/Oct/23 Resolved: 12/Nov/20 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Replication |
| Affects Version/s: | 4.0.18 |
| Fix Version/s: | 4.0.22 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Tess Avitabile (Inactive) | Assignee: | Jason Chan |
| Resolution: | Fixed | Votes: | 1 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||
| Sprint: | Repl 2020-11-02, Repl 2020-11-16 | ||||||||||||
| Participants: | |||||||||||||
| Case: | (copied to CRM) | ||||||||||||
| Description |
|
After the fix inĀ |
| Comments |
| Comment by Jason Chan [ 12/Nov/20 ] |
|
We added two new test suites to v4.0. They are the concurrency_replication_abort_multi_stmt_txn and replica_sets_abort_multi_stmt_txn suites. They use the txn_override_passthrough logic to wrap supported operations in multi-statement transactions. These two suites both set the newly added setTransactionLifetimeToRandomMillis and increaseFrequencyOfPeriodicThreadToExpireTransactions failpoints to set the transactionLifetimeLimitSeconds to a value between 0-20ms and to also have the periodic thread to run every 5ms to check for transaction expiration. All assertion errors are ignored in these suites as the intent is for them to show up as a failure in evergreen only if the system crashes, or if there's a hang or data consistency issue. |
| Comment by Githook User [ 12/Nov/20 ] |
|
Author: {'name': 'Jason Chan', 'email': 'jason.chan@10gen.com', 'username': 'jasonjhchan'}Message: |
| Comment by Tess Avitabile (Inactive) [ 15/Oct/20 ] |
|
I decided to change this ticket to be about diagnosing and fixing the bug on 4.0, instead of adding test coverage for transactionLifetimeLimitSeconds. After the bug is fixed, we should consider adding more test coverage on all branches. |
| Comment by Louis Williams [ 15/Oct/20 ] |
|
|
| Comment by Daniel Gottlieb (Inactive) [ 15/Oct/20 ] |
|
louis.williams I see |