Jean-Daniel Cryans has posted comments on this change. Change subject: csd: validate the CSD using Cloudera public maven repo ......................................................................
Patch Set 1: (2 comments) Thanks for addressing this, was in my TODO list. Please also fix this section: https://github.com/cloudera/kudu/blob/master/java/README.md#building-the-kudu-custom-service-descriptor-csd http://gerrit.cloudera.org:8080/#/c/2311/1/java/pom.xml File java/pom.xml: Line 240: <pluginRepository> Maybe add a comment like above. Line 266: <url>http://maven.jenkins.cloudera.com:8081/artifactory/libs-snapshot-local</url> Unrelated, but do we need this? Actually, both are probably wrong for everyone except Cloudera employees. We should make this cleaner. -- To view, visit http://gerrit.cloudera.org:8080/2311 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I120c874c219abf98e7c33737c34d248daaf8e5bb Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes
