-
Notifications
You must be signed in to change notification settings - Fork 949
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
Parallelize compatibility checks #16251
Parallelize compatibility checks #16251
Conversation
Is this tweaked by |
…parallel operations
Hi @sykhro thanks a lot for checking the PR out. After discussing it with the team, we'll start with just creating a small ThreadPool of say 8, and leave the configuration for a later date. This is mostly because we find the UX could get a bit ugly and some questions were raised as to whether we could harmonize it with the rest of the parallel confs :) |
This reverts commit 883140b.
Changelog: Omit
Docs: Omit
This PR was reverted from 2.4 after finding some issues, the original changelog was
I pulled the current thread count out of thin air after testing some values locally, but as with everything having to do with concurrency, nothing is a best fit for everyone