-
Notifications
You must be signed in to change notification settings - Fork 391
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
Test suite doesn't parallelize as well as it should #7329
Comments
Getting rid of the branch semaphore takes me from 13.5m for the tests (not counting a minute for cached bootstrap) to 11.5. |
msullivan
added a commit
that referenced
this issue
Jun 20, 2024
The test suite used to rely on this for cloning databases, and switching away caused some big slowdowns. Both because this method is somewhat faster but more because the new method only allows one branch at a time to avoid going overbudget on connections. Progress on #7329. On my machine this speed up edb test from 13:49 to 11:20.
msullivan
added a commit
that referenced
this issue
Jun 20, 2024
…#7477) The test suite used to rely on this for cloning databases, and switching away caused some big slowdowns. Both because this method is somewhat faster but more because the new method only allows one branch at a time to avoid going overbudget on connections. Progress on #7329. On my machine this speed up edb test from 13:49 to 11:20.
The |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Some of this is probably because
CREATE DATA BRANCH
is serialized; we could consider loosening that restriction a bit (based on size of conn pool?) or bringing back the old testmode only database clone that uses postgres template databases.Some might be because we have too much stuff with parallelization restrictions, and should tidy it up, and I think our algorithms about allocating those are questionable.
Some might just be inefficiency in the scheduler? I'm not sure.
The text was updated successfully, but these errors were encountered: