Merge pull request #4294 from yandex/optimize-better-error-message

Slightly better message with reason for OPTIMIZE query with "optimize_throw_if_noop"
ClickHouse · Feb 6, 2019 · 1d7b120 · 1d7b120
2 parents 2686f64 + 89861f8
commit 1d7b120
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/dbms/src/Storages/StorageMergeTree.cpp b/dbms/src/Storages/StorageMergeTree.cpp
@@ -403,6 +403,8 @@ bool StorageMergeTree::merge(
             UInt64 max_source_parts_size = merger_mutator.getMaxSourcePartsSize();
             if (max_source_parts_size > 0)
                 selected = merger_mutator.selectPartsToMerge(future_part, aggressive, max_source_parts_size, can_merge, out_disable_reason);
+            else if (out_disable_reason)
+                *out_disable_reason = "Current value of max_source_parts_size is zero";
         }
         else
         {