move side transaction boundaries in OpObserverImpl for preparing multi-document transaction

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Fixed
    • Priority: Major - P3
    • 7.1.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • None
    • Fully Compatible
    • Execution NAMR Team 2023-07-24, Execution NAMR Team 2023-08-07
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      The side transaction currently defined in OpObserverImpl::onTransactionPrepare() makes it a challenge to extract some operations that are not directly related to writing the oplog entry. It may be worthwhile to look into defining the boundaries of the side transaction outside the OpObserverImpl method as an intermediate step to extracting some of these operations into a new OpObserver. For onTransactionPrepare, in production code, there is only one caller in TransactionParticipant, which makes this refactoring somewhat straightforward.

              Assignee:
              Benety Goh
              Reporter:
              Benety Goh
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved: