Merge pull request #455 from ClickHouse/ignore-exceptions-in-client-destructor

slabko · web-flow · commit 3c94b4489419 · 2025-12-11T16:11:44.000+01:00
Ignore exceptions in ~Client
diff --git a/clickhouse/client.cpp b/clickhouse/client.cpp
@@ -289,8 +289,10 @@ Client::Impl::Impl(const ClientOptions& opts,
 }
 
 Client::Impl::~Impl() {
-    // Wrap up an insert if one is in progress.
-    EndInsert();
+    try {
+        EndInsert();
+    } catch (...) {
+    }
 }
 
 void Client::Impl::ExecuteQuery(Query query) {

Original file line number	Diff line number	Diff line change
`@@ -289,8 +289,10 @@ Client::Impl::Impl(const ClientOptions& opts,`
`289`	`289`	`}`
`290`	`290`
`291`	`291`	`Client::Impl::~Impl() {`
`292`		`- // Wrap up an insert if one is in progress.`
`293`		`- EndInsert();`
	`292`	`+ try {`
	`293`	`+ EndInsert();`
	`294`	`+ } catch (...) {`
	`295`	`+ }`
`294`	`296`	`}`
`295`	`297`
`296`	`298`	`void Client::Impl::ExecuteQuery(Query query) {`