-
Notifications
You must be signed in to change notification settings - Fork 69
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
Filter out read-only instances, but skip filtering when it's not required. #565
Conversation
…, because we need to update new flags that rely on read-only modes.
f973823
to
0187781
Compare
I've rerun TODO: test for new |
I've added shardSize=0 cases to |
…ng when number of desired instances is >= number of instances in the zone.
…s enabled and requested shardSize <= 0.
What this PR does:
This PR updates shuffle-sharding to 1) filter out read-only instances on shuffle shard with lookback, if instance changed its state before lookback window, 2) avoid computing shuffle shard if no filtering is required.
Benchmark compared to
main
(updated after 95e9564):Which issue(s) this PR fixes:
Follow up to #560.
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]