[SERVER-50177] snapshot_read_at_cluster_time_crud_operations.js fails with CursorNotFound Created: 07/Aug/20 Updated: 29/Oct/23 Resolved: 16/Dec/20 |
|
| Status: | Closed |
| Project: | Core Server |
| Component/s: | Replication |
| Affects Version/s: | None |
| Fix Version/s: | 4.9.0 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | A. Jesse Jiryu Davis | Assignee: | A. Jesse Jiryu Davis |
| 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 | ||||||||||||
| Sprint: | Repl 2020-12-14, Repl 2020-12-28 | ||||||||||||
| Participants: | |||||||||||||
| Linked BF Score: | 12 | ||||||||||||
| Description |
|
Similar to |
| Comments |
| Comment by Githook User [ 16/Dec/20 ] |
|
Author: {'name': 'A. Jesse Jiryu Davis', 'email': 'jesse@mongodb.com', 'username': 'ajdavis'}Message: |
| Comment by A. Jesse Jiryu Davis [ 13/Dec/20 ] |
|
Reopening the original CR. |
| Comment by Lingzhi Deng [ 20/Nov/20 ] |
|
This makes sense. Thanks for the investigation, Suganthi. |
| Comment by Suganthi Mani [ 20/Nov/20 ] |
|
I think, the original patch in this CR got abandoned due to the below question raised by lingzhi.deng in the CR
When a find/getMore command gets interrupted due to KillOp command after it has generated the batch result(say at line 545for find cmd and line 715 for getMore cmd), the result returned by find/getMore command will have cursor.id in response data. But, when ClientCursorPin destructor is called, the find/getMore notices that it got interrupted which makes it to remove the cursor id from the cursorMap and destroy the cursor. Only the subsequent getMore command will notice that cursor id is not in the cursorMap and would return ErrorCodes::CursorNotFound. So, it's valid for this line in the test snapshot_read_at_cluster_time_crud_operations.js to fail with ErrorCodes::CursorNotFound. Reopening this ticket so that the closed CR can be made active again. |
| Comment by A. Jesse Jiryu Davis [ 10/Aug/20 ] |
|
Closing this ticket and unassigning the BF from myself, perhaps someone else can see the problem from a fresh perspective. |