Noremac201 commented on issue #14096: URL: https://github.com/apache/iceberg/issues/14096#issuecomment-3324892051
Sorry I'm not sure I fully understand - The Thrift API from HMS has had support for `alter_table_with_environment_context` since [rel/1.2.2](https://github.com/apache/hive/blob/rel/release-1.2.2/metastore/if/hive_metastore.thrift#L885-L886). And continues to support it even today in Hive 4.1. If someone has implemented their own `IMetastoreClient` it should support this API, so it should be able to be added for every HMS call syntax-wise. Semantically - the server can choose to leverage the contents of the `environmentContext`, e.x. if you passed the environmentContext with `HIVE-26882` not backported, nothing should happen, but it wouldn't be transactional. Trino was recently providing it for both Hive locks and no locks until Trino version 477 (I've filed https://github.com/trinodb/trino/issues/26656) as well for them. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
