You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2022/03/16 22:20:22 UTC

[GitHub] [hudi] yihua commented on pull request #4695: [WIP][DO NOT MERGE][CI Test Only] Remove hbase-server dependency, pull in HFile related classes, with deps resolution

yihua commented on pull request #4695:
URL: https://github.com/apache/hudi/pull/4695#issuecomment-1069694240


   Per discussion, if we want to pull HFile related code into Hudi, there is more work to do to trim the code that's irrelevant, doing code rewrite inside classes, beyond just pulling in necessary classes, to bring LoC much lower than 66K.  This direction is much more involved.
   
   For now, we'll go with the approach of upgrading HBase to 2.x and properly shading the dependencies, before we write our own file format for the same purpose.  #5004 is ready for review for HBase upgrade to 2.x along this line.  Closing this WIP PR.


-- 
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-unsubscribe@hudi.apache.org

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