Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/12132 )
Change subject: Fix some warnings on GCC7 ...................................................................... Fix some warnings on GCC7 I tried compiling with GCC7 to see what warnings popped up. Fix some ambiguous else warnings resulting from gtest macros. See https://github.com/google/googletest/issues/1119. Add a missing include that broke compilation on the release build. Fix some warnings that detect missing returns when there is a DCHECK (these warnings already occurred in release builds, but they now happen in gcc7 debug builds). Change-Id: I39a12bc5ed6957c147b7f0dba85c7687cc989439 Reviewed-on: http://gerrit.cloudera.org:8080/12132 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/exec/delimited-text-parser-test.cc M be/src/exec/hash-table-test.cc M be/src/exprs/decimal-operators-ir.cc M be/src/exprs/string-functions-ir.cc M be/src/exprs/timezone_db.cc M be/src/runtime/bufferpool/reservation-tracker-test.cc M be/src/runtime/fragment-instance-state.cc M be/src/runtime/fragment-instance-state.h M be/src/service/impala-server.cc M be/src/statestore/statestore.cc M be/src/util/decompress-test.cc M be/src/util/metrics-test.cc M be/src/util/webserver.cc 13 files changed, 38 insertions(+), 17 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/12132 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I39a12bc5ed6957c147b7f0dba85c7687cc989439 Gerrit-Change-Number: 12132 Gerrit-PatchSet: 4 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
