Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

examples: Adds client/server retrying example via service config #7111

Merged
merged 4 commits into from Jun 16, 2020

Conversation

alexanderscott
Copy link
Contributor

Provides an example HelloWorld GRPC client & server which demos the effect of client retry policy configured to the ManagedChannel via GRPC ServiceConfig.

Retry policy configuration is outlined in its proposal, and this example is very similar to the Hedging Example.

I created this example to prove that client GRPC retry via service config does indeed work (although still marked as experimental). Also wrote an example and tests using client future unary calls, but figure this simple example is sufficient.

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jun 10, 2020

CLA Check
The committers are authorized under a signed CLA.

Copy link
Member

@dapengzhang0 dapengzhang0 left a comment

Choose a reason for hiding this comment

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

Can you also add a session here for retry, similar for hedging example?

examples/gradle/wrapper/gradle-wrapper.properties Outdated Show resolved Hide resolved
@alexanderscott
Copy link
Contributor Author

@dapengzhang0 added a section to the README. hope it looks ok 👍

@alexanderscott
Copy link
Contributor Author

any idea why the "Macos" pipeline check could be stuck on "waiting"? I'm unfamiliar with the CI integrations...

Copy link
Member

@dapengzhang0 dapengzhang0 left a comment

Choose a reason for hiding this comment

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

LGTM

examples/README.md Outdated Show resolved Hide resolved
@dapengzhang0
Copy link
Member

dapengzhang0 commented Jun 12, 2020

any idea why the "Macos" pipeline check could be stuck on "waiting"? I'm unfamiliar with the CI integrations...

Those tests are manually triggered by project owners for external user's PRs. No worries.

@dapengzhang0 dapengzhang0 added the kokoro:run Add this label to a PR to tell Kokoro the code is safe and tests can be run label Jun 13, 2020
@grpc-kokoro 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 Jun 13, 2020
@ejona86 ejona86 merged commit a543174 into grpc:master Jun 16, 2020
@ejona86
Copy link
Member

ejona86 commented Jun 16, 2020

@alexanderscott, thank you!

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 12, 2021
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.

None yet

4 participants