Skip to content

Conversation

larry-safran
Copy link
Contributor

RetriableStream was trying to do an optimization by doing early commits when there were not going to be any further explicit retries, but it should never have been done. This is removing the optimization and reversing the test that checked for an early commit.

Fixes #10011

Backport of #10066

…ng. (grpc#10066)

* retries:Remove early commit for transparent retries when no retries or no hedging remain.

Fixes grpc#10011
@larry-safran larry-safran merged commit fcc9286 into grpc:v1.50.x Apr 19, 2023
@larry-safran larry-safran deleted the backport-is10011-1.50 branch April 19, 2023 22:07
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants