Set TransactionRouter::_initiatedCommit to true in TransactionRouter::commitTransaction

XMLWordPrintableJSON

    • Type: Task
    • Resolution: Gone away
    • Priority: Major - P3
    • None
    • Affects Version/s: None
    • Component/s: Sharding
    • None
    • Sharding
    • None
    • 3
    • None
    • None
    • None
    • None
    • None
    • None

      This is just code cleanup blocked on removing the "config server is coordinator" override, but would be easy and quick.

      Rather than setting _initiatedCommit to true in each of

      • TransactionRouter::_commitSingleShardTransaction
      • TransactionRouter::_commitReadOnlyTransaction
      • TransactionRouter::_commitMultiShardTransaction
      • TransactionRouter::_commitWithRecoveryToken

      it can just be set to true in TransactionRouter::commitTransaction. This is blocked on removing the "config server is coordinator" override, because the override checks _initiatedCommit inside TransactionRouter::_commitMultiShardTransaction and expects it to be false the first time commit is called.

            Assignee:
            [DO NOT USE] Backlog - Sharding Team
            Reporter:
            Esha Maharishi (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: