Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22947 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/22947 Reviewed-by: Quanlong Huang <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/service/impala-hs2-server.cc M tests/observability/test_log_fragments.py 2 files changed, 13 insertions(+), 14 deletions(-) Approvals: Quanlong Huang: Looks good to me, approved Impala Public Jenkins: Verified -- 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: merged Gerrit-Change-Id: I93e1fb6c7ba50f47023ca0c382a884093187b847 Gerrit-Change-Number: 22947 Gerrit-PatchSet: 3 Gerrit-Owner: Riza Suminto <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]>
