[SERVER-33640] Blacklist find_and_modify_concurrent_update.js from causal consistency passthrough Created: 02/Mar/18 Updated: 29/Oct/23 Resolved: 04/Mar/18 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Testing Infrastructure |
| Affects Version/s: | None |
| Fix Version/s: | 3.6.4, 3.7.3 |
| Type: | Improvement | Priority: | Major - P3 |
| Reporter: | Charlie Swanson | Assignee: | Charlie Swanson |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||
| Backwards Compatibility: | Fully Compatible | ||||||||
| Backport Requested: |
v3.6
|
||||||||
| Sprint: | Query 2018-03-12 | ||||||||
| Participants: | |||||||||
| Linked BF Score: | 0 | ||||||||
| Description |
|
This test uses a parallel shell, and (in this passthrough) sometimes expects a future read to be causally consistent with the other shell's update. This is not a valid assumption. While we could re-write the test in a way that would allow the main thread to wait and observe the parallel shell's write, it doesn't seem worth it to modify the test and gain that coverage. If such coverage of parallel shells and causal consistency is desired, it should probably be written as it's own standalone test. As part of this, we should do a quick scan to see if there are any other tests that might expect causal consistency with a parallel shell. |
| Comments |
| Comment by Githook User [ 09/Mar/18 ] |
|
Author: {'email': 'charlie.swanson@mongodb.com', 'name': 'Charlie Swanson', 'username': 'cswanson310'}Message: find_and_modify_concurrent_update.js uses startParallelShell to do a (cherry picked from commit c96692173158566c92a78b10df1a6d3d9f570eb1) |
| Comment by Githook User [ 04/Mar/18 ] |
|
Author: {'email': 'charlie.swanson@mongodb.com', 'name': 'Charlie Swanson', 'username': 'cswanson310'}Message: find_and_modify_concurrent_update.js uses startParallelShell to do a |
| Comment by Charlie Swanson [ 02/Mar/18 ] |
|
cc david.storch and max.hirschhorn |