preaudc commented on code in PR #7282: URL: https://github.com/apache/iceberg/pull/7282#discussion_r1163255545
########## docs/hive.md: ########## @@ -127,12 +127,11 @@ The table level configuration overrides the global Hadoop configuration. To use the Tez engine on Hive `3.1.2` or later, Tez needs to be upgraded to >= `0.10.1` which contains a necessary fix [TEZ-4248](https://issues.apache.org/jira/browse/TEZ-4248). - To use the Tez engine on Hive `2.3.x`, you will need to manually build Tez from the `branch-0.9` branch due to a backwards incompatibility issue with Tez `0.10.1`. -You will also need to set the following property in the Hive -configuration: `tez.mrreader.config.update.properties=hive.io.file.readcolumn.names,hive.io.file.readcolumn.ids`. +In both cases, you will also need to set the following property in the `tez-site.xml` configuration +file: `tez.mrreader.config.update.properties=hive.io.file.readcolumn.names,hive.io.file.readcolumn.ids`. Review Comment: Thanks for the review @Fokko ! Sure, is that ok now? -- 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]
