Adar Dembo has posted comments on this change.

Change subject: csd: validate the CSD using Cloudera public maven repo
......................................................................


Patch Set 1:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/2311/1/java/pom.xml
File java/pom.xml:

Line 240:       <pluginRepository>
> Maybe add a comment like above.
Yeah, I meant to, then forgot. Oops.


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 every
They were added to satisfy our internal Java deployment job. I'm not sure if 
they're unnecessary, or how to do it without them. Future work, no doubt.


-- 
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: Adar Dembo <[email protected]>
Gerrit-Reviewer: Jean-Daniel Cryans
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to