-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[Monitoring] Ignore Duplicate Shards #21057
[Monitoring] Ignore Duplicate Shards #21057
Conversation
This eliminates duplicate shards from the shard table by actively ignoring them from the response.
💔 Build Failed |
💔 Build Failed |
💔 Build Failed |
💚 Build Succeeded |
💚 Build Succeeded |
Should we also remove this reference? |
@chrisronline Not in this PR. I am going to backport this PR, so I don't want to have to customize it before backporting (to keep the setting). I did create #21095 for that though. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Functionality wise, I tested that the problem occurs before the code fix and goes away after the code fix.
The code makes sense too!
[Monitoring] Ignore Duplicate Shards This eliminates duplicate shards from the shard table by actively ignoring them from the response.
6.x/6.4: 20c97e5 This did not backport cleanly to 6.3, so I only backported the fix to 6.4. |
This eliminates duplicate shards from the shard table by actively ignoring
them from the response.
Relates to #19704 (comment)