Skip to content

Conversation

@subkanthi
Copy link
Contributor

closes: #616

@github-actions github-actions bot added the INFRA label Jan 15, 2026
@subkanthi subkanthi marked this pull request as draft January 20, 2026 18:19
@subkanthi
Copy link
Contributor Author

The current implementation of commitTable is not atomic, using Hive Locking API similar to iceberg-python

        # https://github.com/apache/hive/blob/master/standalone-metastore/metastore-common/src/main/thrift/hive_metastore.thrift#L1232
        with self._client as open_client:
            lock: LockResponse = open_client.lock(self._create_lock_request(database_name, table_name))

            try:
                if lock.state != LockState.ACQUIRED:
                    if lock.state == LockState.WAITING:
                        self._wait_for_lock(database_name, table_name, lock.lockid, open_client)
                    else:
                        raise CommitFailedException(f"Failed to acquire lock for {table_identifier}, state: {lock.state}")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When is it expected that Hive catalog support will be available?

1 participant