Quanlong Huang has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20925
Change subject: IMPALA-12711: Fix DDL errors are not shown in impalad logs ...................................................................... IMPALA-12711: Fix DDL errors are not shown in impalad logs We use LOG_AND_RETURN_IF_ERROR to log the non-ok status of executing the DDL. However, when enable_async_ddl_execution is true (default), the returned status of ExecDdlRequest() and ExecLoadDataRequest() are about creating the async thread. It's not the error of executing the statement. If the DDL fails, no errors will be shown in impalad logs. This patch fixes it by logging the error when UpdateQueryStatus() is invoked by the async exec thread. A new parameter, 'log_error', is added to this method to control the logging behavior. It's false by default and only set to true when used in the async thread. Tests - Add e2e test to verify the error in logs Change-Id: I8f02f22fa8ebbd2dea722d5586899bf57b66cf40 --- M be/src/service/client-request-state.cc M be/src/service/client-request-state.h M tests/metadata/test_ddl.py 3 files changed, 37 insertions(+), 9 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/25/20925/1 -- To view, visit http://gerrit.cloudera.org:8080/20925 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I8f02f22fa8ebbd2dea722d5586899bf57b66cf40 Gerrit-Change-Number: 20925 Gerrit-PatchSet: 1 Gerrit-Owner: Quanlong Huang <[email protected]>
