elek commented on issue #777: HDDS-3351. Remove unnecessary dependency Curator. URL: https://github.com/apache/hadoop-ozone/pull/777#issuecomment-610295837 Thanks the patch @fapifta (And the quick review @bharatviswa504 and @swagle) For the record: this patch *doesn't* remove the curator dependency. It modifies a `<dependencyManagement>` not a `<dependency>` section. `<depdendencyManagement>` is reponsible for defining the **default versions** not dependencies. > Locally I ran 'mvn clean install -DskipTests', it was running fine. I did the same after this patch (using 9872253e8). And I checked the jar files: ``` mvn clean install -DskipTests cd hadoop-ozone/dist/target/ozone-0.6.0-SNAPSHOT find -name "*cura*" ./share/ozone/lib/curator-client-2.12.0.jar ./share/ozone/lib/curator-framework-2.12.0.jar ./share/ozone/lib/curator-recipes-2.12.0.jar ``` Curator is added as a dependency via the transitive dependencies of Hadoop artifacts. To fully remove it you should `<exclude>` it from the dependency hierarchy. But don't worry. It's a good patch anyway (I am +1, even it if's too late ;-) ). 1. We are not interested in curator, so it can be removed from the `<dependencyManagement>`, too 2. But to remove curator fully we need some exclude rules (the good news is that is should be done by #744, at least on the client side. We should double check the server-side)
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
