Skip to content

add-max-retry-property-support#2062

Merged
pongad merged 4 commits intogoogleapis:masterfrom
AntonZarutsky:add-max-retry-property-support
May 18, 2017
Merged

add-max-retry-property-support#2062
pongad merged 4 commits intogoogleapis:masterfrom
AntonZarutsky:add-max-retry-property-support

Conversation

@AntonZarutsky
Copy link
Copy Markdown

@AntonZarutsky AntonZarutsky commented May 15, 2017

RetrySettings contain property maxAttempt which is not used by Publisher.
Added support for maxAttempt, which gives developer control a bit more over retries.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 15, 2017
@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 779f526 on AntonZarutsky:add-max-retry-property-support into ** on GoogleCloudPlatform:master**.

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 779f526 on AntonZarutsky:add-max-retry-property-support into ** on GoogleCloudPlatform:master**.

@garrettjonesgoogle garrettjonesgoogle requested a review from pongad May 16, 2017 15:42
Copy link
Copy Markdown
Contributor

@pongad pongad left a comment

Choose a reason for hiding this comment

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

Thank you for this! Some style comments, but otherwise LGTM.

}



This comment was marked as spam.

import io.grpc.ManagedChannel;
import io.grpc.Status;
import org.threeten.bp.Duration;

This comment was marked as spam.

This comment was marked as spam.

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 00582b8 on AntonZarutsky:add-max-retry-property-support into ** on GoogleCloudPlatform:master**.

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 49b4e32 on AntonZarutsky:add-max-retry-property-support into ** on GoogleCloudPlatform:master**.

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 49b4e32 on AntonZarutsky:add-max-retry-property-support into ** on GoogleCloudPlatform:master**.

@pongad pongad merged commit 97f6164 into googleapis:master May 18, 2017
chingor13 pushed a commit that referenced this pull request Feb 20, 2026
* fix: add a e2e json test

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
suztomo pushed a commit to suztomo/google-cloud-java that referenced this pull request Mar 23, 2026
meltsufin pushed a commit that referenced this pull request Apr 29, 2026
…yle-plugin to v3.4.0 (#2062)

* build(deps): update dependency org.apache.maven.plugins:maven-checkstyle-plugin to v3.4.0

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
meltsufin pushed a commit that referenced this pull request May 1, 2026
…yle-plugin to v3.4.0 (#2062)

* build(deps): update dependency org.apache.maven.plugins:maven-checkstyle-plugin to v3.4.0

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants