Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10097 )
Change subject: KUDU-2365: [Java] Fix Gradle Artifacts ...................................................................... Patch Set 7: (2 comments) http://gerrit.cloudera.org:8080/#/c/10097/7/java/gradle/publishing.gradle File java/gradle/publishing.gradle: http://gerrit.cloudera.org:8080/#/c/10097/7/java/gradle/publishing.gradle@39 PS7, Line 39: propertyWithDefault > Are the username and password properties required? If so I think this shou They are only required if the maven repository requires them. Below is the error that is printed if you don't set a password and one is required: Execution failed for task ':kudu-client:uploadArchives'. > Could not publish configuration 'archives' > Failed to retrieve remote metadata org.apache.kudu:kudu-client:1.8.0-SNAPSHOT/maven-metadata.xml: Could not transfer metadata org.apache.kudu:kudu-client:1.8.0-SNAPSHOT/maven-metadata.xml from/to remote (https://repository.apache.org/service/local/staging/deploy/maven2): Not authorized , ReasonPhrase:Unauthorized. http://gerrit.cloudera.org:8080/#/c/10097/7/java/kudu-spark/build.gradle File java/kudu-spark/build.gradle: http://gerrit.cloudera.org:8080/#/c/10097/7/java/kudu-spark/build.gradle@22 PS7, Line 22: compile(project(path: ":kudu-client", configuration: "shadow")) { > Can you do the same trick you did in https://github.com/apache/kudu/commit/ I think I could globally exclude it from being shaded. I can't globally exclude it like in the mentioned commit, because that would exclude it all together. -- To view, visit http://gerrit.cloudera.org:8080/10097 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iaef485eca7eebe55286ad5670452839bebb0c47e Gerrit-Change-Number: 10097 Gerrit-PatchSet: 7 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Mon, 23 Apr 2018 21:29:06 +0000 Gerrit-HasComments: Yes
