Todd Lipcon has posted comments on this change.

Change subject: Fixes for flaky-test build
......................................................................


Patch Set 1:

(2 comments)

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

Line 252:     echo Could not fetch flaky tests list.
> Did you test this failure manually? I don't trust "if <complex thing>; then
yea, I set a bad TEST_RESULT_SERVER and made sure the thing exited correctly.


Line 260:     echo No tests are flaky.
> I admire your optimism if you think this code path will ever be taken.
:) it was taken before I added the error check above!


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id6b284e9643fc336e3dbd6a997f0bd60759e97cd
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to