-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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: Deprecate ForwardingChannelBuilder #10587
Conversation
a726446
to
121922d
Compare
@@ -26,7 +26,7 @@ | |||
* using ALTS if applicable and using TLS as fallback. | |||
*/ | |||
public final class ComputeEngineChannelBuilder | |||
extends ForwardingChannelBuilder<ComputeEngineChannelBuilder> { | |||
extends ForwardingChannelBuilder2<ComputeEngineChannelBuilder> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This breaks ABI. We can't change this. Ditto for most of the usages in public APIs. AltsChannelBuilder has few enough users it is probably fine. BinderChannelBuilder would be nice to change, but we may not be easily able to. I'm actually surprised it isn't stable, but I guess it was just BinderServerBuilder that was marked stable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What about AndroidChannelBuilder? That also looks to be stabilized.
Are we going to use AltsChannelBuilder2 & AndroidChannelBuilder2?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Undid:
- AndroidChannelBuilder
- BinderChannelBuilder
- ComputeEngineChannelBuilder
- GoogleDefaultChannelBuilder
121922d
to
2c5c69b
Compare
Undo: - AndroidChannelBuilder - BinderChannelBuilder - ComputeEngineChannelBuilder - GoogleDefaultChannelBuilder Add "As of 1.60.0" to `@deprecated` javadoc tag.
Ready for another round |
When it's approved, I'll edit |
2ae064e
to
d0ec2d9
Compare
Deprecate
ForwardingChannelBuilder
in favor ofForwardingChannelBuilder2