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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
