simplify changes

Author: Varun0157

src/databricks/sql/auth/thrift_http_client.py

@@ -112,6 +112,7 @@ def startRetryTimer(self):
         self.retry_policy and self.retry_policy.start_retry_timer()
 
     def open(self):
+
         # self.__pool replaces the self.__http used by the original THttpClient
         _pool_kwargs = {"maxsize": self.max_connections}
 

src/databricks/sql/backend/sea/backend.py

@@ -285,6 +285,8 @@ def close_session(self, session_id: SessionId) -> None:
             OperationalError: If there's an error closing the session
         """
 
+        logger.debug("SeaDatabricksClient.close_session(session_id=%s)", session_id)
+
         sea_session_id = session_id.to_sea_session_id()
         if sea_session_id is None:
             raise ValueError("Not a valid SEA session ID")

src/databricks/sql/telemetry/telemetry_client.py

@@ -364,7 +364,7 @@ def export_latency_log(self, latency_ms, sql_execution_event, sql_statement_id):
         )
 
     def close(self):
-        """Schedules the client to be closed in the background."""
+        """Schedules client closure."""
         logger.debug(
             "Scheduling background closure for TelemetryClient of connection %s",
             self._session_id_hex,
@@ -376,14 +376,14 @@ def _close_and_wait(self):
         self._flush()
 
         with self._lock:
-            futures_to_wait_on = list(self._pending_futures)
+            pending_events = list(self._pending_futures)
 
-        if futures_to_wait_on:
+        if pending_events:
             logger.debug(
                 "Waiting for %s pending telemetry requests to complete.",
-                len(futures_to_wait_on),
+                len(pending_events),
             )
-            wait(futures_to_wait_on)
+            wait(pending_events)
 
         self._http_client.close()