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

core: Disable retry by default for in-process transport's channel #9361

Merged
merged 1 commit into from
Jul 19, 2022

Conversation

larry-safran
Copy link
Contributor

This mitigates the problem that the buffer limit is not being respected by in-process transport since there is no wire so the bytes aren't actually serialized. Full fix to follow.

See #8712

@ejona86 ejona86 added the TODO:backport PR needs to be backported. Removed after backport complete label Jul 11, 2022
@@ -123,7 +127,7 @@ protected ManagedChannelBuilder<?> delegate() {
}

@Override
public final InProcessChannelBuilder maxInboundMessageSize(int max) {
public InProcessChannelBuilder maxInboundMessageSize(int max) {
Copy link
Member

Choose a reason for hiding this comment

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

Why this change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This change was to eliminate a warning of the redundancy of a final method in a final class.

@larry-safran
Copy link
Contributor Author

larry-safran commented Jul 12, 2022 via email

@larry-safran larry-safran added TODO:backport PR needs to be backported. Removed after backport complete and removed TODO:backport PR needs to be backported. Removed after backport complete labels Jul 18, 2022
@larry-safran
Copy link
Contributor Author

Backport is happening in PR #9368

@larry-safran larry-safran merged commit 7568f8c into grpc:master Jul 19, 2022
larry-safran added a commit to larry-safran/grpc-java that referenced this pull request Jul 19, 2022
larry-safran added a commit that referenced this pull request Jul 21, 2022
temawi pushed a commit to temawi/grpc-java that referenced this pull request Jul 22, 2022
@ejona86 ejona86 removed the TODO:backport PR needs to be backported. Removed after backport complete label Jul 28, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 27, 2022
@larry-safran larry-safran deleted the is-8712 branch February 15, 2023 18:42
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