[SERVER-69749] move packOperationsIntoApplyOps() from OpObserverImpl to TransactionOperations Created: 15/Sep/22  Updated: 29/Oct/23  Resolved: 13/Oct/22

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: 6.2.0-rc0

Type: Improvement 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:
Related
related to SERVER-70092 remove unused param from OpObserver::... Closed
related to SERVER-78675 TransactionParticipant generates appl... Closed
is related to SERVER-68860 add container type to hold replicated... Closed
Backwards Compatibility: Fully Compatible
Sprint: Execution Team 2022-10-17
Participants:

 Description   

The packOperationsIntoApplyOps() is current defined withing the anonymous namespace in the OpObserverImpl implementation. We should look into moving this logic, along with other related functions such as packTransactionStatementsForApplyOps() where possible, into the TransactionOperations class that will be introduced in SERVER-68860.



 Comments   
Comment by Githook User [ 13/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 move getApplyOpsOplogSlotAndOperationAssignmentForTransaction() from OpObserverImpl to TransactionOperations
Branch: master
https://github.com/mongodb/mongo/commit/aaadd6214e5abfe206fcab752fc5ba000909a191

Comment by Githook User [ 13/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 move batched write check out of getApplyOpsOplogSlotAndOperationAssignmentForTransaction()
Branch: master
https://github.com/mongodb/mongo/commit/ad6fa8f4e5dda7b57a05e72efd68e38fb5744029

Comment by Githook User [ 13/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 move multi-doc transaction settings out of packOperationsIntoApplyOps()
Branch: master
https://github.com/mongodb/mongo/commit/1deed7e1caacf7050201f41cf3b58d1b66768c9f

Comment by Githook User [ 12/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 OpObserver::preTransactionPrepare() accepts TransactionOperations
Branch: master
https://github.com/mongodb/mongo/commit/4d9be19d1cf86049fe54e6632950c384b2b7eaa6

Comment by Githook User [ 12/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 OpObserver::preTransactionPrepare() accepts TransactionOperations
Branch: master
https://github.com/10gen/mongo-enterprise-modules/commit/74024124d2f0099607d25ce4d6d463505fbe1a14

Comment by Githook User [ 12/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 OpObserver::onUnpreparedTransactionCommit() accepts TransactionOperations
Branch: master
https://github.com/mongodb/mongo/commit/57746791113c36c0c6a010da2deb580ea3cb97ad

Comment by Githook User [ 12/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 OpObserver::onUnpreparedTransactionCommit() accepts TransactionOperations
Branch: master
https://github.com/10gen/mongo-enterprise-modules/commit/ef63fd1ecde8df3a00fbb281d44cf605eed6641e

Comment by Githook User [ 12/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 TransactionParticipant::retrieveCompletedTransactionOperations() returns TransactionOperations
Branch: master
https://github.com/mongodb/mongo/commit/d6f85073849817aa6224f5c7f0c48f36a3c24ff3

Comment by Githook User [ 12/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 BatchedWriteContext::getBatchedOperations() returns TransactionOperations
Branch: master
https://github.com/mongodb/mongo/commit/c66a4121d166b955523f001a4f81e3ffdabe961d

Comment by Githook User [ 11/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 BatchedWriteContext uses TransactionOperations internally to hold replicated ops
Branch: master
https://github.com/mongodb/mongo/commit/c2e9c636e4864053dbafdbd43756d2721a694e0a

Comment by Githook User [ 11/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 rename TransactionOperations::getMutableOperationsForTransactionParticipant() to getMutableOperationsForOpObserver()
Branch: master
https://github.com/mongodb/mongo/commit/18aeb204e6e1784de0707e7c68ae393884ca791c

Comment by Githook User [ 11/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 add TransactionOperations::getCollectionUUIDs()
Branch: master
https://github.com/mongodb/mongo/commit/66fc26b6731ab140cc1f521c0cf31b1a104dbf79

Comment by Githook User [ 11/Oct/22 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-69749 move TransactionOperations into its own library
Branch: master
https://github.com/mongodb/mongo/commit/da0b9948338552d224f93357c596f8857e69b222

Generated at Thu Feb 08 06:14:17 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.