[SERVER-42335] ShutdownShouldCancelQueuedRequests cpp test can ignore kill signal Created: 22/Jul/19 Updated: 29/Oct/23 Resolved: 30/Jul/19 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Sharding |
| Affects Version/s: | None |
| Fix Version/s: | 4.2.1, 4.3.1 |
| Type: | Bug | Priority: | Minor - P4 |
| Reporter: | Randolph Tan | Assignee: | Randolph Tan |
| 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: |
v4.2
|
||||||||
| Sprint: | Sharding 2019-08-12 | ||||||||
| Participants: | |||||||||
| Linked BF Score: | 19 | ||||||||
| Description |
|
Most of the time test passes because the periodic waitForMajority has not get a chance to consume the queued opTimes before the shutdown sets the promises. However, in some cases, it could be calling the waitForWC, which is a mocked function in the test setup. Because the mock function always returns Status::OK, it will never get the InterruptedByShutdown status code. |
| Comments |
| Comment by Githook User [ 14/Aug/19 ] |
|
Author: {'username': 'renctan', 'email': 'randolph@10gen.com', 'name': 'Randolph Tan'}Message: (cherry picked from commit dc1c7d78c3d5377c79261cebbc6cc70831bbd9a4) |
| Comment by Githook User [ 30/Jul/19 ] |
|
Author: {'name': 'Randolph Tan', 'email': 'randolph@10gen.com', 'username': 'renctan'}Message: |