[SERVER-59676] DocumentSourceFindAndModifyImageLookup can hit invariant when used in a shard Created: 30/Aug/21 Updated: 29/Oct/23 Resolved: 09/Sep/21 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | 5.0.4, 5.1.0-rc0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Randolph Tan | Assignee: | Randolph Tan |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||||||||||||||||||
| Issue Links: |
|
||||||||||||||||||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||||||||||||||||||
| Operating System: | ALL | ||||||||||||||||||||||||
| Backport Requested: |
v5.0
|
||||||||||||||||||||||||
| Sprint: | Sharding 2021-09-06, Sharding 2021-09-20 | ||||||||||||||||||||||||
| Participants: | |||||||||||||||||||||||||
| Story Points: | 1 | ||||||||||||||||||||||||
| Description |
|
and run as an internal client. This is because it relies on getCollectionOptions to get the UUID. However, for internal clients run in the shard, the ShardServerProcessInterface is used. ShardServerProcessInterface uses listCollections to get the options, but it has a special logic that changes how uuid is returned. It appears that the DocumentSourceChangeStreamAddPreImage also gets the UUID the same way. |
| Comments |
| Comment by Githook User [ 07/Oct/21 ] |
|
Author: {'name': 'Randolph Tan', 'email': 'randolph@10gen.com', 'username': 'renctan'}Message: (cherry picked from commit 08651e9e7488dfb76b6fe35efdcc704209eef0f3) |
| 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 [ 09/Sep/21 ] |
|
Author: {'name': 'Randolph Tan', 'email': 'randolph@10gen.com', 'username': 'renctan'}Message: |
| Comment by Randolph Tan [ 30/Aug/21 ] |
|
Attached test.js that must be applied with test.diff that demonstrates this issue. |