Must deep-copy objects in ops array in txn_override.js

XMLWordPrintableJSON

    • Type: Bug
    • Resolution: Gone away
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Replication
    • ALL
    • Repl 2019-01-28, Repl 2019-02-11, Repl 2019-02-25, Repl 2019-03-11, Repl 2019-03-25, Repl 2019-04-08
    • 18
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None
    • None

      When pushing an object onto the 'ops' array in continueTransaction, we currently do not deep-copy it. This means that if a test modifies one of the objects it uses, if we retry the transaction the retry will use the modified objects. We should do a deep copy.

      Note we have a deepCopyObject method in jstests/concurrency/fsm_libs/fsm.js which might be appropriate for this.

              Assignee:
              Judah Schvimer
              Reporter:
              Matthew Russotto
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: