Update comments for clarity

Signed-off-by: Jesse Whitehouse <[email protected]>

Author: Jesse Whitehouse

src/databricks/sql/auth/thrift_http_client.py

@@ -93,6 +93,8 @@ def setCustomHeaders(self, headers: Dict[str, str]):
         super().setCustomHeaders(headers)
 
     def open(self):
+
+        # self.__pool replaces the self.__http used by the original THttpClient
         if self.scheme == "http":
             pool_class = HTTPConnectionPool
         elif self.scheme == "https":

src/databricks/sql/thrift_backend.py

@@ -317,7 +317,10 @@ def attempt_request(attempt):
             try:
                 logger.debug("Sending request: {}".format(request))
                 response = method(request)
+
+                # Calling `close()` here releases the active HTTP connection back to the pool
                 self._transport.close()
+
                 logger.debug("Received response: {}".format(response))
                 return response
             except OSError as err: