Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/12132
to look at the new patch set (#2).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/32/12132/2
--
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: newpatchset
Gerrit-Change-Id: I39a12bc5ed6957c147b7f0dba85c7687cc989439
Gerrit-Change-Number: 12132
Gerrit-PatchSet: 2
Gerrit-Owner: Tim Armstrong <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
