Backport #50811 to 23.4: Don't mark a part as broken on `Poco::Timeou…

…tException`
ClickHouse · Jun 13, 2023 · dcb5228 · dcb5228
1 parent 12d73cd
commit dcb5228
Showing 1 changed file with 9 additions and 0 deletions.
diff --git a/src/Storages/MergeTree/MergeTreeData.cpp b/src/Storages/MergeTree/MergeTreeData.cpp
@@ -95,6 +95,7 @@
 
 #include <fmt/format.h>
 #include <Poco/Logger.h>
+#include <Poco/Net/NetException.h>
 
 template <>
 struct fmt::formatter<DB::DataPartPtr> : fmt::formatter<std::string>
@@ -1245,6 +1246,14 @@ MergeTreeData::LoadPartResult MergeTreeData::loadDataPart(
         res.is_broken = true;
         tryLogCurrentException(log, fmt::format("while loading part {} on path {}", res.part->name, part_path));
     }
+    catch (const Poco::Net::NetException &)
+    {
+        throw;
+    }
+    catch (const Poco::TimeoutException &)
+    {
+        throw;
+    }
     catch (...)
     {
         res.is_broken = true;