thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-19 16:20:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix clang-tidy

Browse Source
This commit is contained in:
Anton Popov 2024-09-04 01:41:55 +00:00
parent 8a5454a548
commit 6042637e67
2 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -8781,7 +8781,7 @@ void incrementMutationsCounters(
Int64 & num_metadata_mutations_to_apply,
const MutationCommands & commands)
{
return updateMutationsCounters(num_data_mutations_to_apply, num_metadata_mutations_to_apply, commands, 1);
updateMutationsCounters(num_data_mutations_to_apply, num_metadata_mutations_to_apply, commands, 1);
}
void decrementMutationsCounters(
@ -8789,7 +8789,7 @@ void decrementMutationsCounters(
Int64 & num_metadata_mutations_to_apply,
const MutationCommands & commands)
{
return updateMutationsCounters(num_data_mutations_to_apply, num_metadata_mutations_to_apply, commands, -1);
updateMutationsCounters(num_data_mutations_to_apply, num_metadata_mutations_to_apply, commands, -1);
}
}

12
src/Storages/MergeTree/MergeTreeDataSelectExecutor.cpp
View File

@ -1550,9 +1550,9 @@ void MergeTreeDataSelectExecutor::selectPartsToRead(
MergeTreeData::DataPartsVector prev_parts;
std::swap(prev_parts, parts);
for (size_t i = 0; i < prev_parts.size(); ++i)
for (const auto & part_or_projection : prev_parts)
{
const auto * part = prev_parts[i]->isProjectionPart() ? prev_parts[i]->getParentPart() : prev_parts[i].get();
const auto * part = part_or_projection->isProjectionPart() ? part_or_projection->getParentPart() : part_or_projection.get();
if (part_values && part_values->find(part->name) == part_values->end())
continue;
@ -1589,7 +1589,7 @@ void MergeTreeDataSelectExecutor::selectPartsToRead(
counters.num_parts_after_partition_pruner += 1;
counters.num_granules_after_partition_pruner += num_granules;
parts.push_back(prev_parts[i]);
parts.push_back(part_or_projection);
}
}
@ -1615,9 +1615,9 @@ void MergeTreeDataSelectExecutor::selectPartsToReadWithUUIDFilter(
MergeTreeData::DataPartsVector prev_parts;
std::swap(prev_parts, selected_parts);
for (size_t i = 0; i < prev_parts.size(); ++i)
for (const auto & part_or_projection : prev_parts)
{
const auto * part = prev_parts[i]->isProjectionPart() ? prev_parts[i]->getParentPart() : prev_parts[i].get();
const auto * part = part_or_projection->isProjectionPart() ? part_or_projection->getParentPart() : part_or_projection.get();
if (part_values && part_values->find(part->name) == part_values->end())
continue;
@ -1667,7 +1667,7 @@ void MergeTreeDataSelectExecutor::selectPartsToReadWithUUIDFilter(
throw Exception(ErrorCodes::LOGICAL_ERROR, "Found a part with the same UUID on the same replica.");
}
selected_parts.push_back(prev_parts[i]);
selected_parts.push_back(part_or_projection);
}
if (!temp_part_uuids.empty())