zhangbutao commented on PR #4573: URL: https://github.com/apache/hive/pull/4573#issuecomment-1759714078
> if all other engines use HiveCatalog, that shouldn't be a problem, isn't it? see #4793 https://github.com/apache/hive/pull/4793 is not related to this change. IMO, other engines, e.g. Trino&Spark also use iceberg metadata file to check cow/mor mode. That is to say, maybe only HS2 use hms propertis to check cow/mor. So I think it is necessary to update iceberg's properties in metadata json file synchronously when writing HMS properties. -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org