Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/12043 )

Change subject: build: enable Java flaky test reporting
......................................................................


Patch Set 7: Verified+1

(4 comments)

Overriding Jenkins, known flaky test failure.

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

http://gerrit.cloudera.org:8080/#/c/12043/2/build-support/jenkins/build-and-test.sh@244
PS2, Line 244:     echo Will retry the flaky tests up to 
$KUDU_FLAKY_TEST_ATTEMPTS times.
> Should KUDU_REPORT_TEST_RESULTS be documented at the top of the script?
Done


http://gerrit.cloudera.org:8080/#/c/12043/2/build-support/jenkins/build-and-test.sh@259
PS2, Line 259: else
             :   CLANG=$(pwd)/thirdparty/clang-toolchain/bin/clang
             : fi
> Now that this is in build-and-test.sh, we can switch on BUILD_TYPE instead
Done


http://gerrit.cloudera.org:8080/#/c/12043/2/build-support/report-test.sh
File build-support/report-test.sh:

http://gerrit.cloudera.org:8080/#/c/12043/2/build-support/report-test.sh@26
PS2, Line 26: # test result server is down.
> Doc up here what environment variables this script expects to find. Maybe e
Done


http://gerrit.cloudera.org:8080/#/c/12043/2/build-support/report-test.sh@30
PS2, Line 30: set -e
> Don't need this anymore.
Done



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ifef74fc9bf5453105c267418fa24daf4c33f73f3
Gerrit-Change-Number: 12043
Gerrit-PatchSet: 7
Gerrit-Owner: Mike Percy <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Fri, 29 Mar 2019 20:03:18 +0000
Gerrit-HasComments: Yes

Reply via email to