Skip to content

Load balancing manual priority#11995

Merged
nikitamikhaylov merged 7 commits intoClickHouse:masterfrom
azat:load_balancing-priority
Jul 6, 2020
Merged

Load balancing manual priority#11995
nikitamikhaylov merged 7 commits intoClickHouse:masterfrom
azat:load_balancing-priority

Conversation

@azat
Copy link
Member

@azat azat commented Jun 27, 2020

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Add replica priority for load_balancing (for manual prioritization of the load balancing).

Detailed description / Documentation draft:
Can be used for:

  • excluding some hosts until there are no errors for other replicas (or you can ignore some errors with distributed_replica_max_ignored_errors)

Fixes: #11565

@blinkov blinkov added the pr-improvement Pull request with some product improvements label Jun 27, 2020
@azat
Copy link
Member Author

azat commented Jun 27, 2020

test_adaptive_granularity/test.py::test_version_update_two_nodes

Is in ci fest

@alexey-milovidov
Copy link
Member

Integration tests

Broken in #11903

@nikitamikhaylov nikitamikhaylov self-assigned this Jun 29, 2020
@azat
Copy link
Member Author

azat commented Jun 29, 2020

Will rebase against uptream/master, since I moved one patch(e02cf7a652eba0e6e72e4dcd92d381727ab9bd12) into #12040 (did not want to create separate PR just for this, but two unrelated changes is worse)

@azat azat force-pushed the load_balancing-priority branch from 1d7408b to 3395276 Compare June 29, 2020 20:03
@azat
Copy link
Member Author

azat commented Jul 2, 2020

Any news on this one? (This will conflicts with #11498 )

@azat
Copy link
Member Author

azat commented Jul 4, 2020

@nikitamikhaylov friendly ping

@nikitamikhaylov
Copy link
Member

@nikitamikhaylov friendly ping

😉

@nikitamikhaylov nikitamikhaylov merged commit 5c6d6bd into ClickHouse:master Jul 6, 2020
@azat azat deleted the load_balancing-priority branch July 6, 2020 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-improvement Pull request with some product improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Extend load_balancing first_or_random to first_2th_or_random

5 participants