[SERVER-22659] Implement commitChunkMigration command on the config server Created: 16/Feb/16 Updated: 05/Jul/16 Resolved: 22/Jun/16 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 3.3.9 |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Kaloian Manassiev | Assignee: | Dianna Hohensee (Inactive) |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||||||||||||||
| Sprint: | Sharding 14 (05/13/16), Sharding 15 (06/03/16), Sharding 16 (06/24/16) | ||||||||||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||||||||||
| Description |
|
Introduce a commitChunkMigration command, only available on the config server, with the following arguments:
This command internally, using a centralized version generation source will produce a new chunk version to be used for the migrated and control chunks, similarly to the logic that is currently used, and updates the config.chunks collection with the new versions, encompassing the writes performs as part of ‘applyOps’ today. Additionally it will return a bson response with the following contents:
One thing to be considered is whether this command should only be made available if the server is running in config server mode. |
| Comments |
| Comment by Githook User [ 22/Jun/16 ] |
|
Author: {u'username': u'DiannaHohensee', u'name': u'Dianna Hohensee', u'email': u'dianna.hohensee@10gen.com'}Message: |
| Comment by Dianna Hohensee (Inactive) [ 16/Jun/16 ] |
|
Depends on WiredTiger will return a WriteConflictException if read from majority snapshot is set on the recovery unit when you go to do a write. So doing reads with the default majority committed snapshot, and then trying to do writes won't work, which it what the CommitChunkMigration command must do. |
| Comment by Dianna Hohensee (Inactive) [ 09/Jun/16 ] |
|
Code review: https://mongodbcr.appspot.com/74000001/ |
| Comment by Githook User [ 01/Jun/16 ] |
|
Author: {u'username': u'DiannaHohensee', u'name': u'Dianna Hohensee', u'email': u'dianna.hohensee@10gen.com'}Message: |
| Comment by Githook User [ 27/May/16 ] |
|
Author: {u'username': u'DiannaHohensee', u'name': u'Dianna Hohensee', u'email': u'dianna.hohensee@10gen.com'}Message: |
| Comment by Githook User [ 23/May/16 ] |
|
Author: {u'username': u'DiannaHohensee', u'name': u'Dianna Hohensee', u'email': u'dianna.hohensee@10gen.com'}Message: |
| Comment by Githook User [ 18/May/16 ] |
|
Author: {u'username': u'DiannaHohensee', u'name': u'Dianna Hohensee', u'email': u'dianna.hohensee@10gen.com'}Message: This commit incidentally fixes a bug in master that |