[GitHub] [hudi] alexeykudinkin edited a comment on pull request #4551: [HUDI-3010] Unbundle parquet-avro and shade hbase in presto-bundle

[GitBox]

alexeykudinkin edited a comment on pull request #4551:
URL: https://github.com/apache/hudi/pull/4551#issuecomment-1013598939


   That makes total sense to me. But for that we have to update the Docker 
images we're using in ITs, right? If'd revert those changes my PR would have 
ITs failing b/c of missing classes. 
   
   Let me know when you'll be able to update Docker images and i'll revert the 
POM changes.
   
   **EDIT** 
   
   Please keep in mind that Hive flows on the current master don't involve 
Metadata table (which uses HFile), and therefore we'd need to validate that it 
works either triggering that flow manually or basing it on top of #4556 which 
does trigger Metadata table usage in Hive flows.


-- 
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: commits-unsubscr...@hudi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

[GitHub] [hudi] alexeykudinkin edited a comment on pull request #4551: [HUDI-3010] Unbundle parquet-avro and shade hbase in presto-bundle

[GitBox]

alexeykudinkin edited a comment on pull request #4551:
URL: https://github.com/apache/hudi/pull/4551#issuecomment-1011791706


   @codope can you please help me understand how Presto would work w/o 
parquet-avro (which RecordReader's are relying on to read using 
ParquetFileReaders) and hbase-shaded-server (which carries HFile, and poised to 
fail if Metadata Table is enabled)


-- 
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: commits-unsubscr...@hudi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org