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 2021/09/23 08:41:34 UTC

[GitHub] [hudi] yanghua commented on pull request #3674: [HUDI-2440] Add dependency change diff script for dependency governace

yanghua commented on pull request #3674:
URL: https://github.com/apache/hudi/pull/3674#issuecomment-925613960


   I received many reports about dependency conflict around hudi, e.g. [here](https://github.com/apache/incubator-kyuubi/pull/994) and Chinese WeChat group. Considering Hudi depends on so many Hadoop ecosystem components. IMO, it's time to do the dependency governance.
   
   In this PR, I provided a dependency utility script that can be used to search/diff dependencies of bundles when the contributors change the dependencys. What's more, I have pre-generated a dependency list for some bundles.
   
   I have two suggestions:
   - build a Github workflow to check the dependency diff, and reviewers should consider any new dependency seriously before merging.
   - based on the pre-generated dependency file to do the dependency governance, you can see there are some dependencies that exist in multiple versions for one bundle.
   
   WDYT? @vinothchandar @xushiyan 


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