[SERVER-70899] move logOplogEntries() from OpObserverImpl to TransactionOperations Created: 27/Oct/22 Updated: 29/Oct/23 Resolved: 15/Dec/22 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 6.3.0-rc0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Benety Goh | Assignee: | Benety Goh |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||||||||||||||||||
| Sprint: | Execution Team 2022-11-14, Execution Team 2022-12-12, Execution Team 2022-11-28, Execution Team 2022-12-26 | ||||||||||||||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||||||||||||||
| Description |
|
This function was introduced to format unprepared multi-document transactions in the OpObserverImpl implementation. It has since been extended to handle prepared transactions and batched writes. As a prerequisite to extending batched writes to span multipl applyOps oplog entries, we should extract this function and add it to the TransactionsOperations for improved testability. This refactor will also affect packTransactionStatementsForApplyOps. |
| Comments |
| Comment by Githook User [ 14/Dec/22 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: The formatting of the applyOps oplog entries is delegated to TransactionOperations |
| Comment by Githook User [ 14/Dec/22 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: This function formats and replicates applyOps oplog entries Replaces logOplogEntries() in op_observer_impl.cpp. |