This repository has been archived by the owner on Sep 26, 2023. It is now read-only.
Is RefreshingManagedChannel being used, or just an experiment? #1084
Assignees
Labels
priority: p2
Moderately-important priority. Fix may not be included in next release.
Comments
miraleung
added
priority: p2
|
Looks like it is, see #804. Please reach out to the original reporters / authors if further discussion is needed. |
|
@miraleung, did you paste the wrong issue/PR? I saw the issue posted and saw the PR #805. But that just exposes an API. It is enabled if you provide a Based on a search of GitHub, it appears it is only used by CBT, but since the implementation is using the grpc-experimental API it requires explicit opt-in via a BetaApi. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
I happened on RefreshingManagedChannel added in #805, and it concerned me because re-creating the channel is heavy-weight and this may prevent future work. I'm most concerned that to my knowledge grpc team wasn't notified, since this can bite us.
Is this being used by someone, or just an experiment?
The text was updated successfully, but these errors were encountered: