thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-15 12:14:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add try catch to data part destructors

Browse Source
This commit is contained in:
alesapin 2024-11-01 13:15:15 +01:00
parent ea806b05a0
commit 24e7fc2714
2 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
src/Storages/MergeTree/MergeTreeDataPartCompact.cpp
View File

@ -256,7 +256,14 @@ bool MergeTreeDataPartCompact::isStoredOnRemoteDiskWithZeroCopySupport() const
MergeTreeDataPartCompact::~MergeTreeDataPartCompact()
{
removeIfNeeded();
try
{
removeIfNeeded();
}
catch (...)
{
tryLogCurrentException(__PRETTY_FUNCTION__);
}
}
}

9
src/Storages/MergeTree/MergeTreeDataPartWide.cpp
View File

@ -241,7 +241,14 @@ bool MergeTreeDataPartWide::isStoredOnRemoteDiskWithZeroCopySupport() const
MergeTreeDataPartWide::~MergeTreeDataPartWide()
{
removeIfNeeded();
try
{
removeIfNeeded();
}
catch (...)
{
tryLogCurrentException(__PRETTY_FUNCTION__);
}
}
void MergeTreeDataPartWide::doCheckConsistency(bool require_part_metadata) const