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

Merge pull request #257 from excitoon/quorum

Browse Source 
StorageReplicatedMergeTree: Correct taking /quorum/last_part into acc…
This commit is contained in:
alexey-milovidov 2016-12-15 19:01:43 +04:00 committed by GitHub
commit 2de93a2a78
1 changed files with 15 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
dbms/src/Storages/StorageReplicatedMergeTree.cpp
View File

@ -1632,6 +1632,21 @@ bool StorageReplicatedMergeTree::canMergeParts(
return false;
}
/// Won't merge last_part even if quorum is satisfied, because we gonna check if replica has this part
/// on SELECT execution.
String quorum_last_part;
if (zookeeper->tryGet(zookeeper_path + "/quorum/last_part", quorum_last_part) && quorum_last_part.empty() == false)
{
ActiveDataPartSet::Part part_info;
ActiveDataPartSet::parsePartName(quorum_last_part, part_info);
if (part_info.left != part_info.right)
throw Exception("Logical error: part written with quorum covers more than one block numbers", ErrorCodes::LOGICAL_ERROR);
if (left->right <= part_info.left && right->left >= part_info.right)
return false;
}
/// Можно слить куски, если все номера между ними заброшены - не соответствуют никаким блокам.
for (Int64 number = left->right + 1; number <= right->left - 1; ++number)
{