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-2444. Remove server side dependencies from ozonefs jar files #130
Conversation
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.
I think we can remove the change added in HDDS-2427 right?
As now ozone-common does not bring in hdds-framework and hadoop-hdds-container-service.
Right, it's not required any more. I added the removal to this patch. |
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. CI issue needs to be fixed.
Not sure if test failures are related to this change.
/retest |
+1. From me too. There is a Rat Check failure. You can fix that push and commit. |
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.
Overall LGTM, just need to fix the License issue and missing newline at EOF.
hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/TestOMStorage.java
Show resolved
Hide resolved
hadoop-ozone/ozone-manager/src/test/java/org/apache/hadoop/ozone/om/TestOMStorage.java
Outdated
Show resolved
Hide resolved
Revert "HDDS-2427. Exclude webapps from hadoop-ozone-filesystem-lib-current uber jar" This reverts commit 6a450fb.
Thanks @bharatviswa504, @anuengineer and @dineshchitlangia the review. The comments are addressed and last build show that the rat is fine. Acceptance tests are unrelated and will be fixed by #282. I will merge this patch soon... |
What changes were proposed in this pull request?
During the review of HDDS-2427 we found that some of the server side dependencies (container-service, framework) are added to the ozonefs library jars. Server side dependencies should be excluded from the client side to make the client safer and the build faster.
This patch doesn't introduce any new features just a dependency cleanup:
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-2444
How was this patch tested?
Compiled and started basic acceptance test (
compose/ozone/test.sh
)