[SERVER-45001] enable commit quorum for two phase index builds by default Created: 06/Dec/19  Updated: 29/Oct/23  Resolved: 04/Mar/20

Status: Closed
Project: Core Server
Component/s: Storage
Affects Version/s: None
Fix Version/s: 4.4.0

Type: Task Priority: Major - P3
Reporter: Benety Goh Assignee: Suganthi Mani
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Backports
Depends
is depended on by SERVER-39485 Add logic for primaries to wait for s... Closed
Problem/Incident
Related
related to SERVER-46668 createIndexes command does not honor ... Closed
related to SERVER-50446 make index builds non-resumable when ... Closed
related to SERVER-49076 Add rollback fuzzer suites to resumab... Closed
related to SERVER-50108 remove enableIndexBuildCommitQuorum s... Backlog
related to SERVER-50602 remove enableIndexBuildCommitQuorum r... Closed
is related to SERVER-46553 Set index build commit quorum default... Closed
is related to SERVER-46675 Convert the the existing "two phase i... Closed
Backwards Compatibility: Fully Compatible
Backport Requested:
v4.4
Sprint: Execution Team 2020-03-09
Participants:
Linked BF Score: 25

 Description   

This sets the default value of the server parameter added in SERVER-44998 so that two phase index builds will run with a commit quorum so that the primary waits for a majority of secondaries to be ready to commit before writing the commitIndexBuild oplog entry.



 Comments   
Comment by Siyuan Zhou [ 06/Mar/20 ]

Yes, they serve the same purpose, but they are separated for a good reason and requires_fcv_44 is exactly designed for this. For example, you'll get the test coverage in 4.6 after we remove requires_fcv_44.

Comment by Suganthi Mani [ 06/Mar/20 ]

siyuan.zhou, it seems both "requires_fcv_44" and "multiversion_incompatible" tag will work due to [this] line in evergreen.yml file for replica_sets_multiversion test suites. I agree, it seems more appropriate to tag with "requires_fcv_44".

Comment by Siyuan Zhou [ 04/Mar/20 ]

suganthi.mani, I found a few more failures on replica_sets_multiversion test suites on this commit on master before the branching. They probably should be tagged with "requires_fcv_44" (not "multiversion_incompatible").

Comment by Githook User [ 04/Mar/20 ]

Author:

{'name': 'Suganthi Mani', 'username': 'smani87', 'email': 'suganthi.mani@mongodb.com'}

Message: SERVER-45001 Ban the replsets jstests that uses 'enableIndexBuildCommitQuorum' server parameter from multiversion suite.
Branch: master
https://github.com/mongodb/mongo/commit/5e607a45d34a4f977341591eec107a7a8a361626

Comment by Githook User [ 03/Mar/20 ]

Author:

{'username': 'smani87', 'name': 'Suganthi Mani', 'email': 'suganthi.mani@mongodb.com'}

Message: SERVER-45001 Enable commit quorum for two phase index builds by default.
Branch: master
https://github.com/mongodb/mongo/commit/cde28e2ab957bd4a27ef240dfbfeea3cc8a70b74

Generated at Thu Feb 08 05:07:35 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.