[SERVER-58509] The transition from nss-based names to uuid-based names of config.cache.chunk.* collections should be majority committed Created: 14/Jul/21  Updated: 29/Oct/23  Resolved: 15/Jul/21

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

Type: Bug Priority: Major - P3
Reporter: Sergi Mateo Bellido Assignee: Antonio Fuschetto
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
Backwards Compatibility: Fully Compatible
Operating System: ALL
Sprint: Sharding EMEA 2021-07-26
Participants:
Linked BF Score: 143

 Description   

The current implementation of ShardServerCatalogCacheLoader::_waitForTasksToCompleteAndRenameChunks doesn't wait for the rename operation to be majority committed. This is a problem because the shard might locally think that the operation was completed: it could happen that the timestamp was majority committed on config.cache.collections but not the renameCollection, so the next time that shard does a refresh it will think that the rename was completed.

After discussing it with pierlauro.sciarelli, we agreed that this method should guarantee the following:

  • If a call to ShardServerCatalogCacheLoader::_waitForTasksToCompleteAndRenameChunks successfully completes, both renameOperation and updateTimestampOnShardCollections should be majority committed.
  • It should support to be called in a scenario in which the target collection already exists but the timestamp is not present on config.cache.collections.*.


 Comments   
Comment by Vivian Ge (Inactive) [ 06/Oct/21 ]

Updating the fixversion since branching activities occurred yesterday. This ticket will be in rc0 when it’s been triggered. For more active release information, please keep an eye on #server-release. Thank you!

Comment by Githook User [ 15/Jul/21 ]

Author:

{'name': 'Antonio Fuschetto', 'email': 'antonio.fuschetto@mongodb.com', 'username': 'afuschetto'}

Message: SERVER-58509 The transition from nss-based names to uuid-based names of config.cache.chunk.* collections should be majority committed
Branch: master
https://github.com/mongodb/mongo/commit/331759573b9e60847a605e4d09292293e7cfa472

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