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

loadbalancer: remove RoundRobinLoadBalancerBuilderProvider and inline its behavior #3137

Conversation

bryce-anderson
Copy link
Contributor

@bryce-anderson bryce-anderson commented Dec 12, 2024

Motivation:

Now that DefaultLoadBalancer is in the servicetalk-loadbalancer
module we no longer need to use a provider in order to migrate
RoundRobinLoadBalancer users to DefaultLoadBalancer.

Modifications:

  • Remove RoundRobinToDefaultLBMigrationProvider.java
  • Use the same flag to switch between DefaultLoadBalancer and
    RoundRobinLoadBalancer in RoundRobinLoadBalancerBuilder.

Motivation:

Now that DefaultLoadBalancer is in the main loadbalancer package
we can avoid using a provider to facilitate the migration.

Modifications:

Remove the migration provider and instead use the types directly.
@bryce-anderson bryce-anderson force-pushed the bl_anderson/DefaultLBMigrationProvider branch from 2454018 to 96f9329 Compare December 19, 2024 22:31
@bryce-anderson bryce-anderson changed the title loadbalancer-experimental: fold the migration provider into the servicetalk-loadbalancer package loadbalancer: remove RoundRobinLoadBalancerBuilderProvider and inline its behavior Dec 19, 2024
@bryce-anderson bryce-anderson marked this pull request as ready for review December 19, 2024 22:49
Copy link
Contributor

@mgodave mgodave left a comment

Choose a reason for hiding this comment

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

LGTM.

@bryce-anderson bryce-anderson merged commit 441c2bc into apple:main Dec 20, 2024
11 checks passed
@bryce-anderson bryce-anderson deleted the bl_anderson/DefaultLBMigrationProvider branch December 20, 2024 00:23
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.

3 participants