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 unnecessary const hinted by clang-tidy

Browse Source 
Removing the reference for Index makes the const unnecessary.
Constness for it is still preserved correctly because Columns
are inmutable pointers to IColumn.
This commit is contained in:
Pablo Marcos 2024-04-27 09:25:44 +02:00
parent 9002d4c751
commit f6b19e143a
3 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -340,7 +340,7 @@ IMergeTreeDataPart::~IMergeTreeDataPart()
}
const IMergeTreeDataPart::Index IMergeTreeDataPart::getIndex() const
IMergeTreeDataPart::Index IMergeTreeDataPart::getIndex() const
{
std::scoped_lock lock(index_mutex);
if (!index_loaded)

2
src/Storages/MergeTree/IMergeTreeDataPart.h
View File

@ -367,7 +367,7 @@ public:
/// Version of part metadata (columns, pk and so on). Managed properly only for replicated merge tree.
int32_t metadata_version;
const Index getIndex() const;
Index getIndex() const;
void setIndex(Index index_);
void unloadIndex();

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

@ -999,7 +999,6 @@ MarkRanges MergeTreeDataSelectExecutor::markRangesFromPKRange(
size_t marks_count = part->index_granularity.getMarksCount();
const auto & index = part->getIndex();
if (marks_count == 0)
return res;