From cfecce10dfa5e21deca68363ba1e613992e1c61e Mon Sep 17 00:00:00 2001 From: Madhavendra Rathore Date: Fri, 26 Dec 2025 17:21:43 +0530 Subject: [PATCH] Fixed the exception handler calls close() on _TelemetryClientHolder objects instead of accessing the client inside them. --- src/databricks/sql/telemetry/telemetry_client.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/databricks/sql/telemetry/telemetry_client.py b/src/databricks/sql/telemetry/telemetry_client.py index 77d1a2f9c..9a38776b1 100644 --- a/src/databricks/sql/telemetry/telemetry_client.py +++ b/src/databricks/sql/telemetry/telemetry_client.py @@ -542,8 +542,8 @@ def _handle_unhandled_exception(cls, exc_type, exc_value, exc_traceback): logger.debug("Handling unhandled exception: %s", exc_type.__name__) clients_to_close = list(cls._clients.values()) - for client in clients_to_close: - client.close() + for holder in clients_to_close: + holder.client.close() # Call the original exception handler to maintain normal behavior if cls._original_excepthook: