change return type of TransactionParticipant::retrieveCompletedOperations() to a pointer

XMLWordPrintableJSON

    • Type: Improvement
    • Resolution: Fixed
    • Priority: Major - P3
    • 6.2.0-rc0
    • Affects Version/s: None
    • Component/s: None
    • None
    • Fully Compatible
    • Execution Team 2022-09-19
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      This function currently returns a non-const reference to the internal vector of operations for a multi-doc transaction. Since the expected usage of this function is to pass along a pointer to the internal operations to the OpObserver methods for committing multi-doc transactions, it would be nice to change the return type of this function from a reference to a pointer.

      Making this change will help avoid accidental copies like the scenarios described in SERVER-69478.

      This is a slight deviation from our code style for function input and outputs.

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

                Created:
                Updated:
                Resolved: