[SERVER-30324] Move logic of removeShard to _configsvrRemoveShard Created: 25/Jul/17 Updated: 30/Oct/23 Resolved: 30/Aug/17 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | 3.5.9 |
| Fix Version/s: | 3.5.12 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Jessica Yu | Assignee: | Hugh Han |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Backwards Compatibility: | Fully Compatible |
| Sprint: | Sharding 2017-07-31, Sharding 2017-08-21 |
| Participants: |
| Description |
|
Logic found in cluster_remove_shard should be moved to _configsvrRemoveShard. Additionally, unit tests using the removeShard logic bundle should be moved out of the shardingCatalogTest file into its own test file, and changed to not go over the mock network for config (but still does for shards). |
| Comments |
| Comment by Githook User [ 15/Aug/17 ] |
|
Author: {'username': 'hughhan1', 'email': 'hughhan1@gmail.com', 'name': 'Hugh Han'}Message: |
| Comment by Githook User [ 08/Aug/17 ] |
|
Author: {'name': 'Mathias Stearn', 'username': 'RedBeard0531', 'email': 'mathias@10gen.com'}Message: Revert " This reverts commit 580cd9ec9e9c5b52d73491655d970d75a7ec1a9c. |
| Comment by Githook User [ 07/Aug/17 ] |
|
Author: {'username': 'hughhan1', 'email': 'hughhan1@gmail.com', 'name': 'Hugh Han'}Message: |