[GitHub] [iceberg] pvary commented on pull request #2163: Hive: Preparation to enable Hive writes with Tez engine

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

pvary commented on pull request #2163:
URL: https://github.com/apache/iceberg/pull/2163#issuecomment-770791526


   +1 from my side, but I would like to see @rdblue's opinion about pushing code which we can not create tests for.
   
   @rdblue: A bit of context: Internally we were able to run write tests with Tez, but there are multiple unreleased fixes needed for Hive and Tez too. The fixes are available on the apache repos but they are not present on any of the releases ATM. Releasing both components would be a slow process because of the dependencies, and would greatly delay adding code here. Shall we wait for those releases or we can put code here which could only be used by patched versions of Hive and Tez?
   
   Thanks,
   Peter
   


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