[GitHub] [iceberg] massdosage commented on pull request #1505: Hive: Make HiveCatalog based tables automatically readable from Hive

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

massdosage commented on pull request #1505:
URL: https://github.com/apache/iceberg/pull/1505#issuecomment-704320460


   > > Thanks @pvary! I've merged this. We should also update the docs to explain when the storage handler will be set so that people can update their hive-site.xml if they choose.
   > 
   > Am I right when I think this should be in https://github.com/apache/iceberg/blob/master/site/docs/hive.md?
   > 
   > Thanks
   > Peter
   
   Yes, I was going to add a section under `Using Hadoop Tables`, something like  `Using Hive Catalog` and this was going to be about finding the Hive table created by Iceberg if you use that method and then how to add the StorageHandler to this table. I think we still need something like that (for existing tables and those created without the config changes in this PR) but we also need a section describing how that can automatically be done at Table creation time from this PR and the `hive-site.xml` etc.


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