[SERVER-31024] Log UUID when collections are renamed Created: 11/Sep/17 Updated: 30/Oct/23 Resolved: 11/May/18 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Replication, Storage |
| Affects Version/s: | None |
| Fix Version/s: | 4.0.0-rc0 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Judah Schvimer | Assignee: | Xiangyu Yao (Inactive) |
| Resolution: | Fixed | Votes: | 1 |
| Labels: | nyc | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Backwards Compatibility: | Fully Compatible | ||||
| Sprint: | Storage NYC 2018-05-07, Storage NYC 2018-05-21 | ||||
| Participants: | |||||
| Linked BF Score: | 0 | ||||
| Description |
|
We've been seeing many bugs with UUIDs that are difficult to debug without actually seeing the UUID to namespace mappings. |
| Comments |
| Comment by Githook User [ 11/May/18 ] |
|
Author: {'name': 'Xiangyu Yao', 'email': 'xiangyu.yao@mongodb.com', 'username': 'xy24'}Message: |
| Comment by Githook User [ 10/May/18 ] |
|
Author: {'name': 'Benety Goh', 'email': 'benety@mongodb.com', 'username': 'benety'}Message: Revert " This reverts commit c23aaecb555518ad8c7a0c8277ad082b19af9154. |
| Comment by Githook User [ 10/May/18 ] |
|
Author: {'name': 'Xiangyu Yao', 'email': 'xiangyu.yao@mongodb.com', 'username': 'xy24'}Message: |
| Comment by Eric Milkie [ 20/Apr/18 ] |
|
renameCollection, got it. |
| Comment by Gregory McKeon (Inactive) [ 20/Apr/18 ] |
|
We were specifically interested in Judah's second comment, which would be great in testing. |
| Comment by Eric Milkie [ 20/Apr/18 ] |
|
Some of this may already be done. We'll look to see if this is complete. |
| Comment by Gregory McKeon (Inactive) [ 19/Apr/18 ] |
|
Sending to storage since we think they see more issues with this than we do. |
| Comment by Judah Schvimer [ 11/Sep/17 ] |
|
Let's also explicitly log when UUIDs change namespace mapping during renameCollection. |
| Comment by Judah Schvimer [ 11/Sep/17 ] |
|
Moving these UUIDCatalog log messages into their own component that always gets logged in our testing could also be useful: https://github.com/mongodb/mongo/blob/19cdcf904e81c70d6a433c76771ff445ae290dcd/src/mongo/db/catalog/uuid_catalog.cpp#L119 |