[SERVER-58385] recovery oplog application updates multikey info on wrong collection Created: 09/Jul/21 Updated: 29/Oct/23 Resolved: 05/Oct/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 4.4.11, 4.4.10, 5.0.4, 5.1.0-rc0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Benety Goh | Assignee: | Benety Goh |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||
| Operating System: | ALL | ||||||||||||||||
| Backport Requested: |
v5.0, v4.4, v4.2, v4.0
|
||||||||||||||||
| Sprint: | Execution Team 2021-07-26 | ||||||||||||||||
| Participants: | |||||||||||||||||
| Linked BF Score: | 70 | ||||||||||||||||
| Description |
|
The list of multikey updates is batched and keyed by collection name during oplog application OplogApplierImpl::_applyOplogBatch. In a recovery scenario where there may be multiple collections with the same name at different points in history, this mapping could be problematic. Downstream components such as the replication StorageInterface::setIndexIsMultikey() would be less prone to looking up the wrong collection from the collection catalog if given the collection UUID. A more robust way to maintain the multikey information to update would be to use the collection UUID rather than the namespace. |
| Comments |
| Comment by Githook User [ 13/Oct/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: (cherry picked from commit 087c24f14c4dca91eade6dff3563649f6edbe6c7) |
| Comment by Githook User [ 11/Oct/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: This test characterizes the current behavior where we still look up the collection by namespace, so the (cherry picked from commit d587a23122d5e64a828eb182bbe3f311541d89a2) |
| Comment by Githook User [ 11/Oct/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: (cherry picked from commit faaca3440968820be133979d503bbe048eb69e4b) |
| Comment by Vivian Ge (Inactive) [ 06/Oct/21 ] |
|
Updating the fixversion since branching activities occurred yesterday. This ticket will be in rc0 when it’s been triggered. For more active release information, please keep an eye on #server-release. Thank you! |
| Comment by Githook User [ 04/Oct/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: Revert " This reverts commit a53ec12efc9f58a56fa7255ffed2e926d33e8269. |
| Comment by Githook User [ 04/Oct/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: (cherry picked from commit faaca3440968820be133979d503bbe048eb69e4b) |
| Comment by Githook User [ 01/Oct/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: (cherry picked from commit 2bb9844342e8c4c3201cdcbb8e84cc0618505ea0) |
| Comment by Githook User [ 20/Sep/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: (cherry picked from commit 087c24f14c4dca91eade6dff3563649f6edbe6c7) |
| Comment by Githook User [ 20/Sep/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: This test characterizes the current behavior where we still look up the collection by namespace, so the (cherry picked from commit d587a23122d5e64a828eb182bbe3f311541d89a2) |
| Comment by Githook User [ 20/Sep/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: (cherry picked from commit faaca3440968820be133979d503bbe048eb69e4b) |
| Comment by Githook User [ 20/Sep/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: (cherry picked from commit 2bb9844342e8c4c3201cdcbb8e84cc0618505ea0) |
| Comment by Githook User [ 20/Jul/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: |
| Comment by Githook User [ 20/Jul/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: This test characterizes the current behavior where we still look up the collection by namespace, so the |
| Comment by Githook User [ 19/Jul/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: |
| Comment by Githook User [ 19/Jul/21 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: |