Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23740 )
Change subject: Fix clean TSAN build. ...................................................................... Fix clean TSAN build. If you run: git clean -xfd (or start with a clean vm). BUILD_TYPE=TSAN ./build-support/jenkins/build-and-test.sh It fails, because thirdparty/installed/uninstrumented/bin/flatc is hardwired in java/kudu-flatbuffers/build.gradle. There is no reason to include the tsan version instead. Solution: Build the uninstrumented version even for TSAN build. Fix clean TSAN build. If you run: git clean -xfd (or start with a clean VM), then BUILD_TYPE=TSAN ./build-support/jenkins/build-and-test.sh it fails because thirdparty/installed/uninstrumented/bin/flatc is hard-wired in java/kudu-flatbuffers/build.gradle. Solution: Build the uninstrumented version even for TSAN builds. Change-Id: Id45e8c98e43df7d769a87b41e50f4781ca234061 Reviewed-on: http://gerrit.cloudera.org:8080/23740 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M java/kudu-flatbuffers/build.gradle M thirdparty/build-thirdparty.sh 2 files changed, 5 insertions(+), 8 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/23740 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id45e8c98e43df7d769a87b41e50f4781ca234061 Gerrit-Change-Number: 23740 Gerrit-PatchSet: 5 Gerrit-Owner: Zoltan Martonka <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]> Gerrit-Reviewer: Zoltan Martonka <[email protected]>
