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

Support for Aggregated-Consistent Hash #121

Merged
merged 2 commits into from
Oct 13, 2020
Merged

Support for Aggregated-Consistent Hash #121

merged 2 commits into from
Oct 13, 2020

Conversation

deniszh
Copy link
Member

@deniszh deniszh commented Jun 7, 2020 •
edited
Loading

@lgtm-com
Copy link

lgtm-com bot commented Jun 7, 2020

This pull request introduces 1 alert when merging 6e3f678 into 87641a5 - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

@lgtm-com
Copy link

lgtm-com bot commented Jun 7, 2020

This pull request introduces 1 alert when merging 44b91fe into 87641a5 - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

@lgtm-com
Copy link

lgtm-com bot commented Jun 7, 2020

This pull request introduces 1 alert when merging 9909adc into 87641a5 - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

@lgtm-com
Copy link

lgtm-com bot commented Jun 7, 2020

This pull request introduces 1 alert when merging 877571b into 87641a5 - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

@lgtm-com
Copy link

lgtm-com bot commented Jun 7, 2020

This pull request introduces 1 alert when merging c87d900 into 87641a5 - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

@lgtm-com
Copy link

lgtm-com bot commented Jun 7, 2020

This pull request introduces 1 alert when merging 245351f into 87641a5 - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

@stale
Copy link

stale bot commented Aug 6, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Aug 6, 2020
@deniszh deniszh added the pinned label Aug 6, 2020
@stale stale bot removed the stale label Aug 6, 2020
@deniszh
Copy link
Member Author

deniszh commented Aug 6, 2020

Still need to finish it...

@deniszh deniszh self-assigned this Aug 6, 2020
@lgtm-com
Copy link

lgtm-com bot commented Oct 3, 2020

This pull request introduces 1 alert when merging c4a7309 into 87641a5 - view on LGTM.com

new alerts:

  • 1 for Wrong number of arguments in a class instantiation

@deniszh
initial backport of graphite-project#56
aa54875 
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
Adding new pythons
7a4ebf9
@deniszh deniszh changed the title initial backport of #56 Support for Aggregated-Consistent Hash Oct 3, 2020
This was referenced Oct 3, 2020
Closed
Open
Closed
Closed
@deniszh deniszh requested a review from piotr1212 October 3, 2020 22:02
@deniszh
Copy link
Member Author

deniszh commented Oct 3, 2020

/cc @ploxlin - could you please review?

@deniszh deniszh requested a review from DanCech October 3, 2020 22:03
@deniszh deniszh merged commit 036c57e into graphite-project:master Oct 13, 2020
@deniszh deniszh deleted the DZ-agg-con-hash branch October 13, 2020 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@piotr1212 piotr1212 Awaiting requested review from piotr1212

@DanCech DanCech Awaiting requested review from DanCech

Assignees

@deniszh deniszh

Labels
pinned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@deniszh