thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-12-01 20:12:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix stupid bug

Browse Source
This commit is contained in:
Alexander Sapin 2023-11-01 13:06:19 +01:00
parent 3a1182f6c0
commit b4328ad8dc
1 changed files with 6 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
src/Storages/MergeTree/MergeTreeData.cpp
View File

@ -1743,9 +1743,9 @@ void MergeTreeData::loadDataParts(bool skip_sanity_checks, std::optional<std::un
if (res.size_of_part) if (res.size_of_part)
{ {
if (unexpected) if (unexpected)
++suspicious_broken_unexpected_parts_bytes; suspicious_broken_unexpected_parts_bytes += *res.size_of_part;
else else
++suspicious_broken_parts_bytes; suspicious_broken_parts_bytes += *res.size_of_part;
} }
} }
else if (res.part->is_duplicate) else if (res.part->is_duplicate)
@ -1792,11 +1792,8 @@ void MergeTreeData::loadDataParts(bool skip_sanity_checks, std::optional<std::un
throw Exception( throw Exception(
ErrorCodes::TOO_MANY_UNEXPECTED_DATA_PARTS, ErrorCodes::TOO_MANY_UNEXPECTED_DATA_PARTS,
"Suspiciously many ({} parts, {} in total) broken parts " "Suspiciously many ({} parts, {} in total) broken parts "
"to remove while maximum allowed broken parts count is {}. You can " "to remove while maximum allowed broken parts count is {}. You can change the maximum value "
"change the maximum value " "with merge tree setting 'max_suspicious_broken_parts' in <merge_tree> configuration section or in table settings in .sql file "
"with merge tree setting 'max_suspicious_broken_parts' "
"in <merge_tree> configuration section or in table settings in "
".sql file "
"(don't forget to return setting back to default value)", "(don't forget to return setting back to default value)",
suspicious_broken_parts, suspicious_broken_parts,
formatReadableSizeWithBinarySuffix(suspicious_broken_parts_bytes), formatReadableSizeWithBinarySuffix(suspicious_broken_parts_bytes),
@ -1807,11 +1804,8 @@ void MergeTreeData::loadDataParts(bool skip_sanity_checks, std::optional<std::un
ErrorCodes::TOO_MANY_UNEXPECTED_DATA_PARTS, ErrorCodes::TOO_MANY_UNEXPECTED_DATA_PARTS,
"Suspiciously big size ({} parts, {} in total) of all broken " "Suspiciously big size ({} parts, {} in total) of all broken "
"parts to remove while maximum allowed broken parts size is {}. " "parts to remove while maximum allowed broken parts size is {}. "
"You can change the maximum value with merge tree setting " "You can change the maximum value with merge tree setting 'max_suspicious_broken_parts_bytes' in <merge_tree> configuration "
"'max_suspicious_broken_parts_bytes' in <merge_tree> " "section or in table settings in .sql file (don't forget to return setting back to default value)",
"configuration "
"section or in table settings in .sql file (don't forget to "
"return setting back to default value)",
suspicious_broken_parts, suspicious_broken_parts,
formatReadableSizeWithBinarySuffix(suspicious_broken_parts_bytes), formatReadableSizeWithBinarySuffix(suspicious_broken_parts_bytes),
formatReadableSizeWithBinarySuffix(settings->max_suspicious_broken_parts_bytes)); formatReadableSizeWithBinarySuffix(settings->max_suspicious_broken_parts_bytes));