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

Merge pull request #71379 from ClickHouse/backport/24.10/71364

Browse Source 
Backport #71364 to 24.10: Add try catch to data part destructors
This commit is contained in:
robot-ch-test-poll1 2024-11-01 21:23:35 +04:00 committed by GitHub
commit b900c37bfb
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 16 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -229,8 +229,15 @@ bool MergeTreeDataPartCompact::isStoredOnRemoteDiskWithZeroCopySupport() const
} }
MergeTreeDataPartCompact::~MergeTreeDataPartCompact() MergeTreeDataPartCompact::~MergeTreeDataPartCompact()
{
try
{ {
removeIfNeeded(); removeIfNeeded();
} }
catch (...)
{
tryLogCurrentException(__PRETTY_FUNCTION__);
}
}
} }

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

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