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

Filter out unhealthy clusters from queue based routing #183

Merged
merged 2 commits into from
Sep 28, 2022
Merged

Filter out unhealthy clusters from queue based routing #183

merged 2 commits into from
Sep 28, 2022

Conversation

akhurana001
Copy link
Contributor

@akhurana001 akhurana001 commented Sep 28, 2022 •
edited
Loading

We generate healthy flag as part of ClusterMonitor but don't use it for routing i.e. still route queries to that cluster.

This filters out unhealthy clusters from queue based routing logic + adds some npe checks.

@akhurana001 akhurana001 merged commit 5a56f9f into master Sep 28, 2022
@akhurana001 akhurana001 deleted the monitor branch September 28, 2022 20:52
@ahackett ahackett mentioned this pull request May 4, 2023
Open
@okayhooni okayhooni mentioned this pull request Aug 12, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@riteshvaryani riteshvaryani riteshvaryani approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@akhurana001 @riteshvaryani