Skip to content

chore(AIP-4221): change example retry policy values#1519

Merged
noahdietz merged 2 commits intoaip-dev:masterfrom
wora:patch-1
Jul 8, 2025
Merged

chore(AIP-4221): change example retry policy values#1519
noahdietz merged 2 commits intoaip-dev:masterfrom
wora:patch-1

Conversation

@wora
Copy link
Copy Markdown
Contributor

@wora wora commented May 30, 2025

The original Google API Design Guide recommended 1s initial delay, which is more sensible than 0.01s. Azure recommends 5s.

2x is the most popular delay multiplier in the industry. As an example, we should use the most popular choice.

60s timeout often leads to thread exhaustion and user frustration and server overloading. Historically, all Google APIs use either 5s or 10s as the deadline. This is especially true for Google REST client libraries.

The original Google API Design Guide recommended 1s initial delay, which is more sensible than 0.01s. Azure recommends 5s.

2 is the most popular delay multiplier in the industry. As an example, we should use the most popular choice.

60s timeout often leads to thread exhaustion and user frustration and server overloading. Historically, all Google APIs use either 5s or 10s as the deadline. This is especially true for Google REST client libraries.
@wora wora requested review from a team as code owners May 30, 2025 12:38
@wora wora requested a review from jskeet May 30, 2025 12:38
@noahdietz noahdietz changed the title Update 4221.md chore(AIP-4221): change example retry policy values Jul 7, 2025
Copy link
Copy Markdown
Collaborator

@noahdietz noahdietz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These values were never intended to be used as guiding examples and this AIP isn't even relevant to API producers - it's a client library AIP with an example of the configuration they'd expect.

That said, the example can be updated.

I'll Approve but leave @jskeet to +2

@noahdietz noahdietz enabled auto-merge (squash) July 8, 2025 17:36
@noahdietz noahdietz merged commit fd73827 into aip-dev:master Jul 8, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants