[SERVER-60819] remove jstests/replsets/rollback_waits_for_bgindex_completion.js Created: 19/Oct/21  Updated: 29/Oct/23  Resolved: 15/Nov/21

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: 5.1.0-rc0
Fix Version/s: 5.2.0

Type: Improvement Priority: Major - P3
Reporter: Adi Zaimi Assignee: Benety Goh
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Duplicate
duplicates SERVER-60428 Investigate effectiveness of test jst... Closed
Related
is related to SERVER-60428 Investigate effectiveness of test jst... Closed
is related to SERVER-30818 Wait for any running background index... Closed
is related to SERVER-44350 write rollback tests for two phase in... Closed
Backwards Compatibility: Fully Compatible
Sprint: Execution Team 2021-11-29
Participants:

 Description   

OLD TITLE: Determine if more test coverage is needed – test jstests/replsets/rollback_waits_for_bgindex_completion.js

We were looking at the effectiveness of jstests/replsets/rollback_waits_for_bgindex_completion.js (SERVER-60428) and we concluded that this test is appropriate only for single-phase index build. The purpose of this ticket is to determine whether more test coverage is needed for two-phase index build and also whether this test should override the default and run with single-phase index build parameter.

More background: This test ensures that when a leader (which is doing an index build) steps down and needs to perform a rollback for an aborted txn, the rollback will wait for the index build to finish. In the case of two-phase index build, rollback code aborts the index build so this test is not applicable. For more info please look at SERVER-60428.



 Comments   
Comment by Githook User [ 15/Nov/21 ]

Author:

{'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}

Message: SERVER-60819 remove rollback_waits_for_bgindex_completion.js
Branch: master
https://github.com/mongodb/mongo/commit/4dad7297741c91c8f8cd67e00ee39e6ec6789f7e

Comment by Benety Goh [ 15/Nov/21 ]

We should be able to remove this test knowing we have coverage introduced in SERVER-44350 for two-phase index builds.

Comment by Benety Goh [ 14/Nov/21 ]

Rollback tests for two phase index builds were added in SERVER-44350.

Comment by Benety Goh [ 11/Nov/21 ]

We added this test in SERVER-30818 as a prerequisite for the Rollback to Timestamp algorithm.

Comment by Adi Zaimi [ 19/Oct/21 ]

To reiterate for clarity: this test is no longer testing single-phase index builds and would need to be modified to test behavior with two-phase index builds.

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