[DRIVERS-2079] Clarify if read command helper should retry getMore, and consider warning Created: 09/Aug/19  Updated: 31/Mar/22

Status: Backlog
Project: Drivers
Component/s: Retryability
Fix Version/s: None

Type: Spec Change Priority: Major - P3
Reporter: Kevin Albertson Assignee: Unassigned
Resolution: Unresolved Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
Driver Changes: Needed

 Description   

The retryable reads spec says:

Any driver that provides generic command runners for read commands(with logic to inherit a client-level read concerns) SHOULD implement retryability for the read-only command runner.

And it explicitly says not to prevent mapReduce from retrying if executed through generic read command helper:

N.B. If mapReduce is executed via a generic command runner for read commands, drivers SHOULD NOT inspect the command to prevent mapReduce from retrying.

But getMore does not have the same note. I think we should be retrying getMore in this case (i.e. not inspecting the command). Though as suggested by shane.harvey perhaps we should update the generic read command docs to warn than getMores will be incorrectly retried when retryReads=true.


Generated at Thu Feb 08 08:24:40 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.