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

      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: