correct TOperationState literal, remove un-necessary check

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

Author: varun-edachali-dbx

src/databricks/sql/backend/thrift_backend.py

@@ -585,8 +585,6 @@ def open_session(self, session_configuration, catalog, schema) -> SessionId:
             response = self.make_request(self._client.OpenSession, open_session_req)
             self._check_initial_namespace(catalog, schema, response)
             self._check_protocol_version(response)
-            if response.sessionHandle is None:
-                return None
             info = (
                 {"serverProtocolVersion": response.serverProtocolVersion}
                 if response.serverProtocolVersion

src/databricks/sql/backend/types.py

@@ -35,7 +35,7 @@ def from_thrift_state(cls, state: ttypes.TOperationState) -> "CommandState":
             return cls.FAILED
         elif state == ttypes.TOperationState.CLOSED_STATE:
             return cls.CLOSED
-        elif state == ttypes.TOperationState.CANCELLED_STATE:
+        elif state == ttypes.TOperationState.CANCELED_STATE:
             return cls.CANCELLED
         else:
             raise ValueError(f"Unknown command state: {state}")