Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16824 )
Change subject: Revert "[thirdparty] Upgrade glog to 0.4.0" ...................................................................... Revert "[thirdparty] Upgrade glog to 0.4.0" This reverts commit 2943aa701ee092158c2084c614a91f92513ef7c4. We are still seeing errors in fresh builds on Centos 7 that look like below with this patch: [ 6%] Running C++ protocol buffer compiler on hash.proto --insertions_out: protoc-gen-insertions: Plugin killed by signal 11. make[2]: *** [src/kudu/util/hash.pb.cc] Error 1 make[1]: *** [src/kudu/util/CMakeFiles/kudu_util_hash_proto.dir/all] Error 2 Change-Id: I5635d1e979be45395d0babdb8c07a234a97a0970 Reviewed-on: http://gerrit.cloudera.org:8080/16824 Tested-by: Grant Henke <[email protected]> Reviewed-by: Bankim Bhavsar <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/fs/dir_util-test.cc M src/kudu/util/logging.h M src/kudu/util/rw_mutex-test.cc M thirdparty/build-definitions.sh M thirdparty/download-thirdparty.sh A thirdparty/patches/glog-fix-symbolization.patch A thirdparty/patches/glog-issue-198-fix-unused-warnings.patch A thirdparty/patches/glog-issue-54-dont-build-tests.patch M thirdparty/patches/glog-support-stacktrace-for-aarch64.patch M thirdparty/vars.sh 10 files changed, 455 insertions(+), 43 deletions(-) Approvals: Grant Henke: Verified Bankim Bhavsar: Looks good to me, approved Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/16824 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I5635d1e979be45395d0babdb8c07a234a97a0970 Gerrit-Change-Number: 16824 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
