Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/15445 )
Change subject: build-and-test.sh: fix TIDY build ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/15445/2//COMMIT_MSG Commit Message: PS2: > Does this mean tidy-bot hasn't been running successfully? What led to the b Without changing the logic I moved this out of the Java build section higher in the script. It seams the Java build was able to cope with the breakage so it went broken for some time. However, the cmake build was more sensitive to an empty JAVA_HOME. At some point we should probably remove this all together. -- To view, visit http://gerrit.cloudera.org:8080/15445 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I19e977c9ddbc25683882ad59766b0e4beb2967e0 Gerrit-Change-Number: 15445 Gerrit-PatchSet: 2 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Mon, 16 Mar 2020 20:27:39 +0000 Gerrit-HasComments: Yes
