[SERVER-74497] Replace usages of getShardVersion for getShardPlacementVersion Created: 01/Mar/23  Updated: 02/Mar/23  Resolved: 02/Mar/23

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

Type: Task Priority: Major - P3
Reporter: Marcos José Grillo Ramirez Assignee: [DO NOT USE] Backlog - Sharding EMEA
Resolution: Won't Do Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Assigned Teams:
Sharding EMEA
Participants:

 Description   

After the changes of PM-2583 the shard version is now a more generalized concept, containing the placement version and the index version. This meant the rename of several usages so classes and functions can match the intent. Considering that these changes won't actually take effect until SERVER-60628 is committed, it didn't made sense to change all usages of getShardVersion at that time, because there is a getShardVersion command running in routers, and they are FCV agnostic, which would cause multiversion testing problems. So the best solution was to provide a new command called getShardPlacementVersion with the appropriate names, and wait until PM-2852 to deprecate the getShardVersion command.

The purpose of this ticket is to change the usages of the getShardVersion command for getShardPlacementVersion in the code base, and deprecate getShardVersion.



 Comments   
Comment by Marcos José Grillo Ramirez [ 02/Mar/23 ]

After talking to the team about this, SERVER-74476 will be re-purposed to add the index version in the getShardVersion command.

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