[SERVER-35174] Add readConcern and readTimestamp to transaction sub-document of currentOp Created: 22/May/18 Updated: 29/Oct/23 Resolved: 25/Jul/18 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Diagnostics, Replication |
| Affects Version/s: | None |
| Fix Version/s: | 4.0.2, 4.1.2 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | William Schultz (Inactive) | Assignee: | Jinny Byun |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | per-txn-metrics, todo_in_code | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||||||||||
| Backport Requested: |
v4.0
|
||||||||||||||||||||||||||||
| Sprint: | Repl 2018-06-18, Repl 2018-07-02, Repl 2018-07-16, Repl 2018-07-30 | ||||||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||||||
| Description |
|
We want the currentOp entry for both active and inactive transactions to contain an entry with a transaction.parameters sub document that contains the readConcern and the readTimestamp of the transaction. The readConcern we output here should be the readConcern after any potential upconvert, which should be stored in ReadConcernArgs::_level. The read timestamp should be stored on the session. For active transactions, the CurOp::reportCurrentOpForClient method should attach this information to the appropriate sub-document, and for inactive transactions, the Session::reportStashedState method should attach it. |
| Comments |
| Comment by Githook User [ 15/Aug/18 ] |
|
Author: {'username': 'jinichu', 'email': 'jinnybyun@gmail.com', 'name': 'jinichu'}Message: (cherry picked from commit 925cfae906b8b335bd657752d4b6a9297dafaaca) |
| Comment by Githook User [ 25/Jul/18 ] |
|
Author: {'username': 'jinichu', 'name': 'jinichu', 'email': 'jinnybyun@gmail.com'}Message: |