[SERVER-46060] Add test that FCV 4.4 config server expects commitChunkMigration to contain a ChunkVersion Created: 10/Feb/20 Updated: 29/Oct/23 Resolved: 05/Mar/20 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 4.4.0-rc0, 4.7.0 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Esha Maharishi (Inactive) | Assignee: | Alexander Taskov (Inactive) |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Backwards Compatibility: | Fully Compatible | ||||
| Backport Requested: |
v4.4
|
||||
| Sprint: | Sharding 2020-03-09 | ||||
| Participants: | |||||
| Description |
|
This ticket is to add test coverage for this assertion in ShardingCatalogManager::checkChunkMatchesRequest. It could be done through a unit test or integration test. We have unit these two related unit test files: that call methods on ShardingCatalogManager, so it might make sense to create an analogous sharding_catalog_manager_move_chunks_test.cpp. Alternatively, it could be tested in an integration test by directly sending commitChunkMigration to the config server without a ChunkVersion when the config server is in FCV 4.4. SeeĀ this class that represents a commiitChunkMigration request. |
| Comments |
| Comment by Max Hirschhorn [ 01/Apr/20 ] |
|
The changes from 3a4b19a were backported to the 4.4 branch together along with the changes from 3cbe84a from Author: {'name': 'Alex Taskov', 'username': 'alextaskov', 'email': 'alex.taskov@mongodb.com'}Message: (cherry picked from commit 3cbe84a6a68a87942a9e040c44a4cc41f46f4e86) |
| Comment by Githook User [ 05/Mar/20 ] |
|
Author: {'username': 'alextaskov', 'name': 'Alex Taskov', 'email': 'alex.taskov@mongodb.com'}Message: |