close_command return not used, replacing with None and logging resp

Signed-off-by: varun-edachali-dbx <[email protected]>

Author: varun-edachali-dbx

src/databricks/sql/backend/databricks_client.py

@@ -50,7 +50,7 @@ def cancel_command(self, command_id: CommandId) -> None:
         pass
 
     @abstractmethod
-    def close_command(self, command_id: CommandId) -> ttypes.TStatus:
+    def close_command(self, command_id: CommandId) -> None:
         pass
 
     @abstractmethod

src/databricks/sql/backend/thrift_backend.py

@@ -1224,15 +1224,17 @@ def cancel_command(self, command_id: CommandId) -> None:
         req = ttypes.TCancelOperationReq(thrift_handle)
         self.make_request(self._client.CancelOperation, req)
 
-    def close_command(self, command_id: CommandId):
+    def close_command(self, command_id: CommandId) -> None:
         thrift_handle = command_id.to_thrift_handle()
         if not thrift_handle:
             raise ValueError("Not a valid Thrift command ID")
 
         logger.debug("ThriftBackend.close_command(command_id=%s)", command_id)
         req = ttypes.TCloseOperationReq(operationHandle=thrift_handle)
         resp = self.make_request(self._client.CloseOperation, req)
-        return resp.status
+        logger.debug(
+            "ThriftBackend.close_command(command_id=%s) -> %s", command_id, resp
+        )
 
     def handle_to_id(self, session_id: SessionId) -> Any:
         """Get the raw session ID from a SessionId"""