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

add support for aggregated consistent hashing #56

Closed

Conversation

klynch
Copy link

@klynch klynch commented Jun 12, 2015

You can configure the relay method to be one of "consistent-hashing" or "aggregated-consistent-hashing". If omitted, "consistent-hashing" is used by default. Use of "aggregated-consistent-hashing" usually requires a rules file to be provided to relevant commands. This can be provided on the command line.

@deniszh deniszh closed this Aug 3, 2016
@deniszh deniszh reopened this Aug 3, 2016
@deniszh
Copy link
Member

deniszh commented Aug 3, 2016

@klynch - Tests are broken completely, not ready to merge, unfortunately.
Is this change still relevant / needed?

@deniszh
Copy link
Member

deniszh commented Aug 6, 2016

Closing for now

@deniszh deniszh closed this Aug 6, 2016
@deniszh deniszh reopened this May 13, 2020
@deniszh deniszh added the pinned label May 13, 2020
@deniszh deniszh self-assigned this May 13, 2020
deniszh added a commit to deniszh/carbonate that referenced this pull request Jun 7, 2020
deniszh added a commit to deniszh/carbonate that referenced this pull request Oct 3, 2020
ah

lgtm is wrong here

migrating to monkeypatching

lgtm is wrong here

default relay method

style fixes

fix linting

init tests

different aggregation_rules init

Adding gitignore

Fixing tests
deniszh added a commit to deniszh/carbonate that referenced this pull request Oct 3, 2020
ah

lgtm is wrong here

migrating to monkeypatching

lgtm is wrong here

default relay method

style fixes

fix linting

init tests

different aggregation_rules init

Adding gitignore

Fixing tests

Adding new pythons

Adding new pythons
@deniszh
Copy link
Member

deniszh commented Oct 3, 2020

Closed in favor of #121

@deniszh deniszh closed this Oct 3, 2020
@deniszh deniszh removed the pinned label Oct 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants