thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-26 09:32:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #61466 from vitlibar/fix-finishing-failed-restore

Browse Source 
Fix finishing a failed RESTORE
This commit is contained in:
Alexey Milovidov 2024-03-16 13:41:33 +03:00 committed by GitHub
commit f31beb98cd
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/Backups/RestorerFromBackup.cpp
View File

@ -101,10 +101,12 @@ RestorerFromBackup::RestorerFromBackup(
RestorerFromBackup::~RestorerFromBackup() RestorerFromBackup::~RestorerFromBackup()
{ {
if (!futures.empty()) /// If an exception occurs we can come here to the destructor having some tasks still unfinished.
/// We have to wait until they finish.
if (getNumFutures() > 0)
{ {
LOG_ERROR(log, "RestorerFromBackup must not be destroyed while {} tasks are still running", futures.size()); LOG_INFO(log, "Waiting for {} tasks to finish", getNumFutures());
chassert(false && "RestorerFromBackup must not be destroyed while some tasks are still running"); waitFutures();
} }
} }