Aggarwal-Raghav opened a new pull request, #6667: URL: https://github.com/apache/hive/pull/6667
### What changes were proposed in this pull request? Add support for the Iceberg `REWRITE_MANIFESTS` procedure in Hive https://iceberg.apache.org/docs/latest/spark-procedures/#rewrite_manifests The implementation is very similar to Trino's approach https://github.com/trinodb/trino/blob/master/plugin/trino-iceberg/src/main/java/io/trino/plugin/iceberg/procedure/OptimizeManifests.java ### Why are the changes needed? Optimize manifest files using hive engine rather than running spark sql `CALL spark_catalog.system.rewrite_manifests(...)`. Also, hive `EXPIRE_SNAPSHOTS` or `OPTIMIZE TABLE` doesn't clean up manifest files. ### Does this PR introduce _any_ user-facing change? Yes. A new SQL command is now available for Hive-Iceberg users: ```ALTER TABLE <table> EXECUTE REWRITE_MANIFESTS``` ### How was this patch tested? Using UT. Added `testRewriteManifests` and `testRewriteManifestsPartitioned` in `TestHiveIcebergSnapshotOperations` -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
