Xuebin Su has uploaded a new patch set (#3). ( 
http://gerrit.cloudera.org:8080/21581 )

Change subject: IMPALA-13115: Add query id to message of UNREACHABLE_IMPALADS
......................................................................

IMPALA-13115: Add query id to message of UNREACHABLE_IMPALADS

This patch adds the query id to the message of error
UNREACHABLE_IMPALADS so that user can easily figure out the affected
queries.

This patch also adds a helper method `get_error_formatter()` to
translate error message in the error code definition to Python-
style format string so that we don't have to do it manually.

Testing:
- Added an assertion to the case of killing and restarting a worker
  in the custom cluster test to ensure that the error message is in
  both the query profile web page and the log got by the client.

Change-Id: I72260687ad395f97972e65ffcb47846e9de75bfb
---
M be/src/service/impala-server.cc
M common/thrift/generate_error_codes.py
M tests/common/impala_test_suite.py
M tests/custom_cluster/test_process_failures.py
4 files changed, 47 insertions(+), 24 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/81/21581/3
--
To view, visit http://gerrit.cloudera.org:8080/21581
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I72260687ad395f97972e65ffcb47846e9de75bfb
Gerrit-Change-Number: 21581
Gerrit-PatchSet: 3
Gerrit-Owner: Xuebin Su <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>

Reply via email to