massdosage commented on pull request #1104: URL: https://github.com/apache/iceberg/pull/1104#issuecomment-641991343
Hey there, some context on why #933 was paused for a month - in developing it we noticed issues with Guava dependencies required by Hive so we focused on #935 and #1067 which have subsequently been merged and proven to work for #933 (and within a Hive client). In the meantime we continued development of additional features for the InputFormat as well as a SerDe and a StorageHandler for Hive, all of which are temporarily at https://github.com/ExpediaGroup/hiveberg. We are now in the process of trying to merge these changes into master via #933 and #1103. Then we'd like to add more of the integration tests from Hiveberg and ideally tidy up and refactor more of the code between the two input formats. I'd have a preference for merging any improvements that you have here into #933 or subsequent follow-up PRs so hopefully we end up with the best of both worlds and a really feature-rich, well-tested InputFormat. @rdblue @rdsr what are your thoughts? ---------------------------------------------------------------- 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]
