[DRIVERS-2477] change stream consecutive resume test should use write concern majority Created: 18/Oct/22 Updated: 27/Oct/23 Resolved: 24/Oct/22 |
|
| Status: | Closed |
| Project: | Drivers |
| Component/s: | Change Streams |
| Fix Version/s: | None |
| Type: | Task | Priority: | Unknown |
| Reporter: | Bailey Pearson | Assignee: | Bailey Pearson |
| Resolution: | Gone away | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Driver Compliance: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Description |
SummaryThe change stream consecutive resume test (here) fails intermittently in Node because the test inserts documents with write the default write concern instead of majority. Change events are not reported by the server until they are majority committed. The test inserts multiple documents and attempts to iterate the change stream, expecting change events. If the documents haven't been committed by the time the test attempts to iterate the stream, no change will be reported and the test will fail. This only impacts the consecutive resume test and does not impact users because the change event will eventually be reported. This test specifically sets up fail commands on getMores, so that the getMore fail and the subsequent aggregate call succeeds and returns a change event in the firstBatch. The race condition is present only because we expect the aggregate to immediately return documents in the firstBatch. MotivationWho is the affected end user?Drivers engineers. How does this affect the end user?Flakey change stream tests. How likely is it that this problem or use case will occur?Unsure. This started failing recently in Node and fails consistently now. If the problem does occur, what are the consequences and how severe are they?There are no consequences outside of a failing test. Is this issue urgent?No. Is this ticket required by a downstream team?No. Is this ticket only for tests?Only for tests. |
| Comments |
| Comment by Benji Rewis (Inactive) [ 20/Oct/22 ] |