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

Remove unused Strings from MergeTreeData

Browse Source 
Compiler does not emit a warning when there is an unused variable of
a type with a non-trivial constructor. std::string is one of them.
Hence, these two variables were overlooked. This commit removes them
This commit is contained in:
Mikhail Koviazin 2023-11-28 16:33:44 +02:00
parent de0876ed68
commit daaa575106
No known key found for this signature in database
GPG Key ID: 0EEAA1BF0787792F
1 changed files with 0 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -4793,7 +4793,6 @@ void MergeTreeData::movePartitionToDisk(const ASTPtr & partition, const String &
if (parts.empty())
{
String no_parts_to_move_message;
if (moving_part)
throw Exception(ErrorCodes::UNKNOWN_DISK, "Part '{}' is already on disk '{}'", partition_id, disk->getName());
else
@ -4875,7 +4874,6 @@ void MergeTreeData::movePartitionToVolume(const ASTPtr & partition, const String
if (parts.empty())
{
String no_parts_to_move_message;
if (moving_part)
throw Exception(ErrorCodes::UNKNOWN_DISK, "Part '{}' is already on volume '{}'", partition_id, volume->getName());
else