Skip to content

api: add support for SocketAddress types in ManagedChannelProvider#9076

Merged
sanjaypujare merged 3 commits intogrpc:masterfrom
sanjaypujare:udf-change
Apr 22, 2022
Merged

api: add support for SocketAddress types in ManagedChannelProvider#9076
sanjaypujare merged 3 commits intogrpc:masterfrom
sanjaypujare:udf-change

Conversation

@sanjaypujare
Copy link
Copy Markdown
Contributor

also add support for SocketAddress types in NameResolverProvider
Use scheme in target URI to select a NameRseolverProvider and get
that provider's supported SocketAddress types.
implement selection in ManagedChannelRegistry of appropriate
ManagedChannelProvider based on NameResolver's SocketAddress types

fixes #3085

also add support for SocketAddress types in NameResolverProvider
Use scheme in target URI to select a NameRseolverProvider and get
that provider's supported SocketAddress types.
implement selection in ManagedChannelRegistry of appropriate
ManagedChannelProvider based on NameResolver's SocketAddress types
@sanjaypujare sanjaypujare requested a review from ejona86 April 10, 2022 23:13
Comment thread api/src/main/java/io/grpc/ManagedChannelRegistry.java Outdated
Comment thread api/src/main/java/io/grpc/ManagedChannelRegistry.java Outdated
Comment thread api/src/main/java/io/grpc/ManagedChannelRegistry.java Outdated
Comment thread api/src/main/java/io/grpc/ManagedChannelRegistry.java Outdated
Comment thread api/src/main/java/io/grpc/NameResolverProvider.java Outdated
Comment thread api/src/main/java/io/grpc/NameResolverProvider.java Outdated
@sanjaypujare
Copy link
Copy Markdown
Contributor Author

@ejona86 addressed all the comments. Thanks

@sanjaypujare sanjaypujare merged commit 538db03 into grpc:master Apr 22, 2022
@sanjaypujare sanjaypujare deleted the udf-change branch April 22, 2022 16:10
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Jul 22, 2022
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.

Allow choosing the ManagedChannelProvider based on the NameResolver

2 participants