grpc-core: use retryThrottling from defaultServiceConfig #11290

JoaoVitorStein · 2024-06-14T13:01:52Z

Context: #11274

linux-foundation-easycla · 2024-06-14T13:01:59Z

The committers listed above are authorized under a signed CLA.

✅ login: JoaoVitorStein / name: João Vitor Stein (8167d8a, 50f1c44, 9e88706)

core/src/test/java/io/grpc/internal/ManagedChannelImplTest.java

core/src/main/java/io/grpc/internal/ManagedChannelImpl.java

…icy from defaultServiceConfig

JoaoVitorStein · 2024-07-01T12:44:19Z

@ejona86 Got the CLA approval

ejona86 · 2024-07-03T15:19:27Z

Thank you!

grpc-core: use retryThrottling from defaultServiceConfig

8167d8a

JoaoVitorStein commented Jun 14, 2024

View reviewed changes

core/src/test/java/io/grpc/internal/ManagedChannelImplTest.java Show resolved Hide resolved

grpc-core: use retryThrottling even if serviceConfigLookup=false

50f1c44

ejona86 reviewed Jun 25, 2024

View reviewed changes

core/src/main/java/io/grpc/internal/ManagedChannelImpl.java Outdated Show resolved Hide resolved

grpc-core: remove unneded check for nulls when setting throttling pol…

9e88706

…icy from defaultServiceConfig

kannanjgithub approved these changes Jun 26, 2024

View reviewed changes

ejona86 approved these changes Jul 2, 2024

View reviewed changes

ejona86 added the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Jul 2, 2024

grpc-kokoro removed the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Jul 2, 2024

ejona86 merged commit 062ebb4 into grpc:master Jul 3, 2024
12 of 13 checks passed

ejona86 mentioned this pull request Jul 3, 2024

Retry throttling not being respected with default service config #11274

Closed

github-actions bot locked as resolved and limited conversation to collaborators Oct 2, 2024

Provide feedback