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
android: final stabilization of AndroidChannelBuilder #6097
android: final stabilization of AndroidChannelBuilder #6097
Conversation
@@ -388,94 +388,4 @@ public void enterIdle() { | |||
enterIdleCount++; | |||
} | |||
} | |||
|
|||
private static class ScheduledExecutorImpl implements ScheduledExecutorService { |
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.
How does this change relate?
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 is something in the test that I (unfortunately) forgot to delete last time when deleting the related test methods.
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.
Do that in a separate PR. If this API change was reverted, we wouldn't want to revert the deletion.
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.
Sure, it's in #6098. (And sorry for the force-push...)
@@ -51,7 +50,6 @@ | |||
* | |||
* @since 1.12.0 | |||
*/ | |||
@ExperimentalApi("https://github.com/grpc/grpc-java/issues/6043") |
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.
Go ahead and copy this to the @Deprecated
method. In general, we can't remove deprecated methods. We can only remove this one because it was never stabilized.
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.
Done.
1e7067f
to
4129de5
Compare
Resolves #6043.
Upon this point,
AndroidChannelBuilder
is considered to be stabilized (fromBuidler(...)
is deprecated and will be kept for a couple of releases).