From 06f0267ed142d76109b2d6d48d3c413bdaf0fc17 Mon Sep 17 00:00:00 2001
From: Alexey Korepanov <alexey.korepanov@clickhouse.com>
Date: Sun, 25 Aug 2024 11:27:43 +0200
Subject: [PATCH] Do not show statistics if exception happens

---
 ...JSONCompactWithProgressRowOutputFormat.cpp | 22 +++++++++----------
 1 file changed, 10 insertions(+), 12 deletions(-)

diff --git a/src/Processors/Formats/Impl/JSONCompactWithProgressRowOutputFormat.cpp b/src/Processors/Formats/Impl/JSONCompactWithProgressRowOutputFormat.cpp
index 0814e5f45d6..cb3398196f4 100644
--- a/src/Processors/Formats/Impl/JSONCompactWithProgressRowOutputFormat.cpp
+++ b/src/Processors/Formats/Impl/JSONCompactWithProgressRowOutputFormat.cpp
@@ -117,18 +117,16 @@ void JSONCompactWithProgressRowOutputFormat::writeProgress()
 
 void JSONCompactWithProgressRowOutputFormat::finalizeImpl()
 {
-    JSONUtils::writeCompactAdditionalInfo(
-        row_count,
-        statistics.rows_before_limit,
-        statistics.applied_limit,
-        statistics.watch,
-        statistics.progress,
-        settings.write_statistics && exception_message.empty(),
-        *ostr);
-
-    if (!exception_message.empty())
-    {
-        writeCString("\n", *ostr);
+    if (exception_message.empty()) {
+        JSONUtils::writeCompactAdditionalInfo(
+            row_count,
+            statistics.rows_before_limit,
+            statistics.applied_limit,
+            statistics.watch,
+            statistics.progress,
+            settings.write_statistics && exception_message.empty(),
+            *ostr);
+    } else {
         JSONUtils::writeCompactObjectStart(*ostr);
         JSONUtils::writeException(exception_message, *ostr, settings, 0);
         JSONUtils::writeCompactObjectEnd(*ostr);