Hello Tim Armstrong, Impala Public Jenkins,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/12414

to look at the new patch set (#2).

Change subject: IMPALA-8177: Log DDL failures in coordinator logs
......................................................................

IMPALA-8177: Log DDL failures in coordinator logs

If a DDL fails for some reason, it helps to log the failure message in
the coordinator logs so that we can differentiate between failed and
successful DDL queries.

For ex:

[0d66cd6004b4:21000] default> drop database foo;
Query: drop database foo
ERROR: ImpalaRuntimeException: Error making 'dropDatabase' RPC to Hive 
Metastore:
CAUSED BY: NoSuchObjectException: foo
----- coordinator logs -------
I0208 12:13:26.251695 25474 Frontend.java:1242] 
704e86fff482c0b5:f0501f9400000000] Analyzing query: drop database foo
I0208 12:13:26.253773 25474 Frontend.java:1282] 
704e86fff482c0b5:f0501f9400000000] Analysis finished.
I0208 12:13:26.419946 25474 client-request-state.cc:176] 
704e86fff482c0b5:f0501f9400000000] ImpalaRuntimeException: Error making 
'dropDatabase' RPC to Hive Metastore:
CAUSED BY: NoSuchObjectException: foo
I0208 12:13:26.419992 25474 impala-server.cc:1142] 
704e86fff482c0b5:f0501f9400000000] UnregisterQuery(): 
query_id=704e86fff482c0b5:f0501f9400000000
I0208 12:13:26.419997 25474 impala-server.cc:1249] 
704e86fff482c0b5:f0501f9400000000] Cancel(): 
query_id=704e86fff482c0b5:f0501f9400000000
-------------------------------

Testing: Verified manually by running a few DDLs that fail and then inspecting
the coordinator log file.

Change-Id: Ie89291ee27156c701e07cea44ad3ee07ec54ab42
---
M be/src/common/status.h
M be/src/service/client-request-state.cc
2 files changed, 11 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/14/12414/2
--
To view, visit http://gerrit.cloudera.org:8080/12414
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie89291ee27156c701e07cea44ad3ee07ec54ab42
Gerrit-Change-Number: 12414
Gerrit-PatchSet: 2
Gerrit-Owner: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>

Reply via email to