Hello Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/22947
to look at the new patch set (#2).
Change subject: IMPALA-14097: Fix test_log_fragments.py
......................................................................
IMPALA-14097: Fix test_log_fragments.py
test_log_fragments.py broken after HS2 switch in IMPALA-13916. The test
look for log line from ImpalaServer::QueryToTQueryContext in
impala-beeswax-server.cc.
VLOG_QUERY << "query: " << ThriftDebugString(query);
This log line does not appear when using HS2 protocol. The equivalent
log line in impala-hs2-server.cc is in ImpalaServer::ExecuteStatement
VLOG_QUERY << "ExecuteStatement(): request=" << ...
This patch also remove redundant TExecuteStatementReq logging in
ImpalaServer::TExecuteStatementReqToTQueryContext. Either of
ImpalaServer::ExecuteStatement or ImpalaServer::ExecutePlannedStatement
has log the same TExecuteStatementReq already.
Testing:
- Run and pass test_log_fragments.py by itself.
Change-Id: I93e1fb6c7ba50f47023ca0c382a884093187b847
---
M be/src/service/impala-hs2-server.cc
M tests/observability/test_log_fragments.py
2 files changed, 13 insertions(+), 14 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/47/22947/2
--
To view, visit http://gerrit.cloudera.org:8080/22947
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I93e1fb6c7ba50f47023ca0c382a884093187b847
Gerrit-Change-Number: 22947
Gerrit-PatchSet: 2
Gerrit-Owner: Riza Suminto <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
