[SERVER-48601] ChunkSplitter should use the same chunk bounds for splitVector and splitChunk Created: 04/Jun/20 Updated: 29/Oct/23 Resolved: 29/Jul/20 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 4.4.1, 4.7.0, 4.2.10 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Matthew Saltz (Inactive) | Assignee: | Jack Mulrow |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | sharding-wfbf-day | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Operating System: | ALL | ||||||||
| Backport Requested: |
v4.4, v4.2
|
||||||||
| Sprint: | Sharding 2020-07-13, Sharding 2020-07-27, Sharding 2020-08-10 | ||||||||
| Participants: | |||||||||
| Linked BF Score: | 5 | ||||||||
| Description |
|
The ChunkSplitter takes in min/max values for the chunk range as input parameters, but then looks the chunk up again in the routing info based on the min key. The min/max for that chunk are used in the call to splitVector, but the call to splitChunk uses the input parameters. The problem is that in between the time the chunk was submitted for splitting and the time that we look up the chunk again in the routing table, the routing table could have changed and these values could be different, leading to weird outcomes. |
| Comments |
| Comment by Githook User [ 12/Aug/20 ] |
|
Author: {'name': 'Jack Mulrow', 'email': 'jack.mulrow@mongodb.com', 'username': 'jsmulrow'}Message: (cherry picked from commit 578ebc515122de8ed50b2b914d9d92ee61139b3f) |
| Comment by Githook User [ 12/Aug/20 ] |
|
Author: {'name': 'Jack Mulrow', 'email': 'jack.mulrow@mongodb.com', 'username': 'jsmulrow'}Message: (cherry picked from commit 578ebc515122de8ed50b2b914d9d92ee61139b3f) |
| Comment by Githook User [ 29/Jul/20 ] |
|
Author: {'name': 'Jack Mulrow', 'email': 'jack.mulrow@mongodb.com', 'username': 'jsmulrow'}Message: |
| Comment by Matthew Saltz (Inactive) [ 04/Jun/20 ] |
|
I'll reassign this to Backlog - Sharding user when it's available. FYI ratika.gandhi |