[GitHub] [iceberg] rdblue commented on pull request #1751: Set Input/Output formats for the HMS storage descriptor

[GitBox <gi...@apache.org>]

rdblue commented on pull request #1751:
URL: https://github.com/apache/iceberg/pull/1751#issuecomment-725593828


   @boroknagyz, the problem with setting this is that Hive doesn't include these classes by default and missing classes will break Hive. We've added configuration to set up the Hive storage descriptor properly to both the Hive configuration (`iceberg.engine.hive.enabled`) and to table properties (`engine.hive.enabled`). If you enable Hive either way, then the `IcebergHiveStorageHandler` is set up.
   
   I also would not recommend any engine use the Hive configuration other than Hive. I think Impala is planning to add native read support for Iceberg tables, which would be a much better way to read.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org