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

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

    XMLWordPrintableJSON

Details

    • Icon: Task Task
    • Resolution: Gone away
    • Icon: Major - P3 Major - P3
    • None
    • None
    • Sharding
    • None
    • Sharding

    Description

      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.

      Attachments

        Activity

          People

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

            Dates

              Created:
              Updated:
              Resolved: