-
Notifications
You must be signed in to change notification settings - Fork 7k
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
Use multiple threads while reading the metadata of tables to restore #60040
Use multiple threads while reading the metadata of tables to restore #60040
Conversation
This is an automated comment for commit 0df2a64 with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page Successful checks
|
2fc0f34
to
a04de77
Compare
a04de77
to
3a9819b
Compare
3a9819b
to
b1e2b8d
Compare
@SmitaRKulkarni Please take a look |
@vitlibar |
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.
Rest all LGTM
I don't think we should provide user a choice here, because using multiple threads seems to be faster and as correct as it was. |
Test failures: |
Changelog category:
Changelog entry:
Use multiple threads while reading the metadata of tables from a backup while executing the RESTORE command.
Before this PR the metadata of tables was always read from a backup in a single thread. That was slow if the number of tables in a backup was high.