Mike Percy has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8662 )

Change subject: java build: Add script to verify output of generated JARs
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/8662/4/build-support/jenkins/build-and-test.sh
File build-support/jenkins/build-and-test.sh:

http://gerrit.cloudera.org:8080/#/c/8662/4/build-support/jenkins/build-and-test.sh@402
PS4, Line 402:     $SOURCE_ROOT/build-support/verify_jars.pl .
> I think you could run the script just once if you move it below the gradle
I'm concerned that it will mask issues that only occur in the Maven build. Or 
do the Gradle artifacts get dropped elsewhere?



--
To view, visit http://gerrit.cloudera.org:8080/8662
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie16bca12dfe2d0267e559c91abe1178d906e3122
Gerrit-Change-Number: 8662
Gerrit-PatchSet: 4
Gerrit-Owner: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-Comment-Date: Tue, 28 Nov 2017 20:42:43 +0000
Gerrit-HasComments: Yes

Reply via email to