[SERVER-43901] Add test coverage in shard_key_util.cpp Created: 08/Oct/19 Updated: 27/Oct/23 Resolved: 25/May/23 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Question | Priority: | Major - P3 |
| Reporter: | Randolph Tan | Assignee: | Marcos José Grillo Ramirez |
| Resolution: | Works as Designed | Votes: | 0 |
| Labels: | sharding-common-backlog, sharding-wfbf-day | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Assigned Teams: |
Sharding EMEA
|
| Sprint: | Sharding EMEA 2023-05-15, Sharding EMEA 2023-05-29 |
| Participants: |
| Description |
|
Add test or delete dead code: |
| Comments |
| Comment by Marcos José Grillo Ramirez [ 25/May/23 ] |
|
After taking a look this code was changed by SERVER-54587 and it is in use in the shardCollection path, and tested by a jstest, so there is no need to do any work for this ticket. |
| Comment by Connie Chen [ 21/Dec/22 ] |
|
The work in this ticket is to review whether we actually still use this code. |