New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HDDS-1113. Remove default dependencies from hadoop-ozone project #490
Conversation
I thought we fixed this, but I get this error when I build with this patch applied.
|
|
💔 -1 overall
This message was automatically generated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 lgtm.
Thanks @arp7 the review, I am merging it to the trunk, right now. |
…ersionUpgrade. Prevent invalid state updations on barrier. * Introduced a additional barrier state NEW. * Add state validations before updating the barrier. * Fix existing TestZkBarrier tests that are disabled and add new tests to verify the intended behavior. Author: Shanthoosh Venkataraman <svenkataraman@linkedin.com> Author: Shanthoosh Venkataraman <santhoshvenkat1988@gmail.com> Reviewers: Jagadish <jagadish@apache.org> Closes apache#490 from shanthoosh/fix_barrier_state_transitions
There are two ways to define common dependencies with maven:
1.) put all the dependencies to the parent project and inherit them
2.) get all the dependencies via transitive dependencies
TLDR; I would like to switch from 1 to 2 in hadoop-ozone
My main problem with the first approach that all the child project get a lot of dependencies independent if they need them or not. Let's imagine that I would like to create a new project (for example a java csi implementation) It doesn't need ozone-client, ozone-common etc, in fact it conflicts with ozone-client. But these jars are always added as of now.
Using transitive dependencies is more safe: we can add the dependencies where we need them and all of the other dependent projects will use them.
See: https://issues.apache.org/jira/browse/HDDS-1113