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.

Sign up for GitHub

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

Jump to bottom

Update default lb-method to be random two least_conn #391

Merged
merged 1 commit into from
Oct 11, 2018
Merged

Update default lb-method to be random two least_conn #391

merged 1 commit into from
Oct 11, 2018
+7 −7

Conversation

Dean-Coakley
Copy link
Contributor

Proposed changes

  • Random two least_conn is now the default lb-method as implemented in 5c7be23

Checklist

Before creating a PR, run through this checklist and mark each as complete.

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have checked that all unit tests pass after adding my changes
  • I have updated necessary documentation
  • I have rebased my branch onto master
  • I will ensure my PR is targeting the master branch and pulling from my branch from my own fork

@Dean-Coakley Dean-Coakley added the enhancement Pull requests for new features/feature enhancements label Oct 9, 2018
@Dean-Coakley Dean-Coakley self-assigned this Oct 9, 2018
@Dean-Coakley
Update default lb-method to: random two least_conn
c96907d 
* Random two least_conn is now the default lb-method as implemented in 5c7be23
@Dean-Coakley Dean-Coakley merged commit 3f0740d into master Oct 11, 2018
@Dean-Coakley Dean-Coakley deleted the default-random-leastconn branch October 11, 2018 18:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@isaachawley isaachawley isaachawley approved these changes

@pleshakov pleshakov pleshakov approved these changes

Assignees

@Dean-Coakley Dean-Coakley

Labels
enhancement Pull requests for new features/feature enhancements
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Dean-Coakley @isaachawley @pleshakov