Uploaded image for project: 'Core Server'
  1. Core Server
  2. SERVER-40219

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

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

      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:
            backlog-server-sharding [DO NOT USE] Backlog - Sharding Team
            Reporter:
            esha.maharishi@mongodb.com Esha Maharishi (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: