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

fix: Adding service provider configuration files to fix the grpc sink… #203

Merged
merged 1 commit into from
Nov 17, 2022

Conversation

Shreyansh228
Copy link
Contributor

@Shreyansh228 Shreyansh228 commented Nov 17, 2022

… exception "Could not find policy 'pick_first'". Issue number here.

In the current implementation, the gRPC client chooses default LoadBalancerProvider ('pick_first') and default NameResolverProvider(DNS). The implementation classes PickFirstLoadBalancerProvider and DnsNameResolverProvider respectively are missing.

We can handle the issue with the including implementation classes through service provider configuration files.

  • SPI io.grpc.LoadBalancerProvider: io.grpc.internal.PickFirstLoadBalancerProvider
  • SPI io.grpc.NameResolverProvider: io.grpc.internal.DnsNameResolverProvider

Hey, I just made a Pull Request!

Please describe what you added, and add a screenshot if possible.
That makes it easier to understand the change so we can :shipit: faster.

✔️ Checklist

  • A changelog describing the changes and affected packages.
  • Added or updated documentation
  • Tests for new functionality and regression tests for bug fixes
  • Screenshots attached (for UI changes)

… exception "Could not find policy 'pick_first'"

In the current implementation, the gRPC client chooses default LoadBalancerProvider ('pick_first') and default NameResolverProvider(DNS). The implementation classes PickFirstLoadBalancerProvider and DnsNameResolverProvider respectively are missing.

We can handle the issue with the including implementation classes through service provider configuration files.
* SPI io.grpc.LoadBalancerProvider: io.grpc.internal.PickFirstLoadBalancerProvider
* SPI io.grpc.NameResolverProvider: io.grpc.internal.DnsNameResolverProvider
Copy link
Collaborator

@sumitaich1998 sumitaich1998 left a comment

Choose a reason for hiding this comment

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

lgtm

@sumitaich1998 sumitaich1998 merged commit 4237ead into main Nov 17, 2022
@sumitaich1998 sumitaich1998 deleted the fix-grpc-loadbalance-provider branch November 17, 2022 09:39
lavkesh pushed a commit that referenced this pull request Nov 22, 2022
… exception "Could not find policy 'pick_first'" (#203)

In the current implementation, the gRPC client chooses default LoadBalancerProvider ('pick_first') and default NameResolverProvider(DNS). The implementation classes PickFirstLoadBalancerProvider and DnsNameResolverProvider respectively are missing.

We can handle the issue with the including implementation classes through service provider configuration files.
* SPI io.grpc.LoadBalancerProvider: io.grpc.internal.PickFirstLoadBalancerProvider
* SPI io.grpc.NameResolverProvider: io.grpc.internal.DnsNameResolverProvider

Co-authored-by: Shreyansh <shreyansh.sangolli@gojek.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants