From 2c06eb8382633ed1bd25bae63cf81ca6e469ac0f Mon Sep 17 00:00:00 2001 From: alesapin Date: Mon, 29 Mar 2021 14:59:28 +0300 Subject: [PATCH] Relax covering part check on mutation --- src/Storages/StorageReplicatedMergeTree.cpp | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/src/Storages/StorageReplicatedMergeTree.cpp b/src/Storages/StorageReplicatedMergeTree.cpp index 0849f65477d..195730b3548 100644 --- a/src/Storages/StorageReplicatedMergeTree.cpp +++ b/src/Storages/StorageReplicatedMergeTree.cpp @@ -1663,9 +1663,10 @@ bool StorageReplicatedMergeTree::tryExecutePartMutation(const StorageReplicatedM if (source_part->name != source_part_name) { - throw Exception("Part " + source_part_name + " is covered by " + source_part->name - + " but should be mutated to " + entry.new_part_name + ". This is a bug.", - ErrorCodes::LOGICAL_ERROR); + LOG_WARNING("Part " + source_part_name + " is covered by " + source_part->name + + " but should be mutated to " + entry.new_part_name + ". " + + "Possibly the mutation of this part is not needed and will be skipped. This shouldn't happen often."); + return false; } /// TODO - some better heuristic?