From 6047db7ea2d9f013c26aaf744cf2d4f996a1b857 Mon Sep 17 00:00:00 2001 From: Alexey Zatelepin Date: Mon, 7 May 2018 23:29:13 +0300 Subject: [PATCH] fix removing failed quorum parts at startup (tryGetChilden actually returns error code, not success flag) --- .../Storages/MergeTree/ReplicatedMergeTreeRestartingThread.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dbms/src/Storages/MergeTree/ReplicatedMergeTreeRestartingThread.cpp b/dbms/src/Storages/MergeTree/ReplicatedMergeTreeRestartingThread.cpp index 37ef004dd55..925e02b599f 100644 --- a/dbms/src/Storages/MergeTree/ReplicatedMergeTreeRestartingThread.cpp +++ b/dbms/src/Storages/MergeTree/ReplicatedMergeTreeRestartingThread.cpp @@ -245,7 +245,7 @@ void ReplicatedMergeTreeRestartingThread::removeFailedQuorumParts() auto zookeeper = storage.getZooKeeper(); Strings failed_parts; - if (!zookeeper->tryGetChildren(storage.zookeeper_path + "/quorum/failed_parts", failed_parts)) + if (zookeeper->tryGetChildren(storage.zookeeper_path + "/quorum/failed_parts", failed_parts) != ZooKeeperImpl::ZooKeeper::ZOK) return; for (auto part_name : failed_parts)