[SERVER-45381] Replace all references to "retriable" with "retryable" Created: 06/Jan/20  Updated: 06/Dec/22

Status: Open
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: None

Type: Improvement Priority: Trivial - P5
Reporter: Jeremy Mikola Assignee: Backlog - Service Architecture
Resolution: Unresolved Votes: 1
Labels: neweng
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Assigned Teams:
Service Arch
Participants:

 Description   

SERVER-42908 makes reference to a kAllRetriableErrors identifier, which is inconsistent with all of our public-facing APIs and documentation where "retryable" is used. Assuming there's no BC concerns with doing so, I think it'd be prudent to revise this to use "retryable" in order to minimize the chance that a server developer sees this and uses the alternative spelling down the line in something that does end up in a public-facing API.

cc: emily.giurleo, who caught this while working on DRIVERS-525



 Comments   
Comment by Lauren Lewis (Inactive) [ 21/Dec/21 ]

We haven’t heard back from you in at least 1 year, so I'm going to close this ticket. If this is still an issue for you, please provide additional information and we will reopen the ticket.

Comment by Jeremy Mikola [ 10/Jan/20 ]

While reading SPEC-1505, I realized there is also a RetriableError category in the error codes file. I'm not sure if this is derivative of the kAllRetriableErrors identifier above, but if not we should add it to the list of things to rename.

Comment by Danny Hatcher (Inactive) [ 06/Jan/20 ]

As we made the conscious decision to use the proposed spelling in Retryable Writes, I think it's a fair ask.

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