[SERVER-26562] Move CommitChunkMigration command implementation into a Catalog function and unit test it Created: 10/Oct/16 Updated: 05/Apr/17 Resolved: 09/Dec/16 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 3.4.2, 3.5.2 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Dianna Hohensee (Inactive) | Assignee: | Nathan Myers |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Backwards Compatibility: | Fully Compatible | ||||
| Backport Requested: |
v3.4
|
||||
| Sprint: | Sharding 2016-10-31, Sharding 2016-11-21, Sharding 2016-12-12, Sharding 2017-01-02 | ||||
| Participants: | |||||
| Description |
|
While working on As part of this ticket, the code from the CommitChunkMigration command should be moved to be a commitChunkMigration method on ShardingCatalogManager with the following parameters:
The current parameters fromShardCollectionVersion and shardHasDistributedLock will be removed in place of the exact chunk version being transmitted and the newly written unit-tests. In addition to the change above, unit-tests must be implemented similar to what we do for split and merge. The following test case should be included:
|
| Comments |
| Comment by Githook User [ 20/Jan/17 ] |
|
Author: {u'username': u'nathan-myers-mongo', u'name': u'Nathan Myers', u'email': u'nathan.myers@10gen.com'}Message: (cherry picked from commit 9b403d89bb81064a9d4813724d075c51b0121e97) |
| Comment by Nathan Myers [ 12/Dec/16 ] |
|
Improve testing of 3.4 releases. |
| Comment by Githook User [ 08/Dec/16 ] |
|
Author: {u'username': u'nathan-myers-mongo', u'name': u'Nathan Myers', u'email': u'nathan.myers@10gen.com'}Message: |
| Comment by Githook User [ 05/Dec/16 ] |
|
Author: {u'username': u'Machyne', u'name': u'Matt Cotter', u'email': u'matt.cotter@mongodb.com'}Message: |
| Comment by Githook User [ 05/Dec/16 ] |
|
Author: {u'username': u'nathan-myers-mongo', u'name': u'Nathan Myers', u'email': u'nathan.myers@10gen.com'}Message: |
| Comment by Nathan Myers [ 23/Nov/16 ] |
|
We will remove the MoveChunkRequest::_takeDistLock member and corresponding arguments and message fields for 3.5 after completing the backport to 3.4. Includes CommitChunkMigrationRequest::appendAsCommand argument shardHasDistributedLock. |