[SERVER-64809] Remove performViewChange flag for shardsvrCollModParticipant command Created: 22/Mar/22 Updated: 26/Oct/23 |
|
| Status: | Backlog |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Rui Liu | Assignee: | Backlog - Catalog and Routing |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | oldshardingemea, shardingemea-qw | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Assigned Teams: |
Catalog and Routing
|
| Participants: | |
| Story Points: | 2 |
| Description |
|
Currently performViewChange is a flag sent from collMod coordinator to instruct whether shards should update view definitions as part of the command. This flag is set to true only for the primary shard, since only primary shard holds the view catalog. We can improve this by doing the check on the shard. |
| Comments |
| Comment by Rui Liu [ 24/Mar/22 ] |
|
cris.insignarescuello Yes this ticket is a refactoring work. The current solution works but it would be nicer if we can remove this flag and do the primary shard check locally on the shard. This means we could be able to send the same command to all shards in one batch, hence removing this block of code. I put the ticket to sharding since it probably requires knowledge on how to detect shard being primary in a safe way and also DDL coordinators are in general owned by the sharding team. This is probably a low-priority task though. sergi.mateo-bellido and tommaso.tocci helped me on the collMod coordinator and has more background on this. |