[SERVER-32131] ChangeStreams lookup_post_image.js test makes assumptions that don't hold up in secondary read passthrough Created: 30/Nov/17 Updated: 30/Oct/23 Resolved: 07/Dec/17 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Aggregation Framework, Replication |
| Affects Version/s: | None |
| Fix Version/s: | 3.6.1, 3.7.1 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Spencer Brody (Inactive) | Assignee: | Matthew Russotto |
| 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: |
v3.6
|
||||||||||||||||
| Sprint: | Repl 2017-12-18 | ||||||||||||||||
| Participants: | |||||||||||||||||
| Linked BF Score: | 0 | ||||||||||||||||
| Description |
|
The lookup_post_image.js test does an update to a document, then immediately removes it, the advances a changeStream to consume the notification corresponding to the update and expects the looked up fullDocument from the updateLookup to be null since the doc was deleted. This assertion is valid if the read and write go to the same node, but in the change_streams_secondary_reads suite this assumption is invalid, leading to the test to be flaky. We should either modify the test to not make this incorrect assumption, or blacklist it from the change_streams_secondary_reads suite. |
| Comments |
| Comment by Githook User [ 12/Dec/17 ] |
|
Author: {'name': 'Matthew Russotto', 'email': 'matthew.russotto@10gen.com', 'username': 'mtrussotto'}Message: (cherry picked from commit 1fd23dfafd2127c00cf47ae4d5acef75d6546def) |
| Comment by Githook User [ 07/Dec/17 ] |
|
Author: {'name': 'Matthew Russotto', 'username': 'mtrussotto', 'email': 'matthew.russotto@10gen.com'}Message: |