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

fix comment

Browse Source
This commit is contained in:
tavplubix 2021-04-29 20:36:55 +03:00 committed by GitHub
parent 4bf9b37435
commit f5ba30e4c0
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Storages/StorageReplicatedMergeTree.cpp
View File

@ -1549,14 +1549,14 @@ bool StorageReplicatedMergeTree::tryExecuteMerge(const LogEntry & entry)
if (!source_part_or_covering)
{
/// We does not have one of source parts locally, try to take some already merged part from someone.
/// We do not have one of source parts locally, try to take some already merged part from someone.
LOG_DEBUG(log, "Don't have all parts for merge {}; will try to fetch it instead", entry.new_part_name);
return false;
}
if (source_part_or_covering->name != source_part_name)
{
/// We does not have source part locally, but we have some covering part. Possible options:
/// We do not have source part locally, but we have some covering part. Possible options:
/// 1. We already have merged part (source_part_or_covering->name == new_part_name)
/// 2. We have some larger merged part which covers new_part_name (and therefore it covers source_part_name too)
/// 3. We have two intersecting parts, both cover source_part_name. It's logical error.