[ 
https://issues.apache.org/jira/browse/IMPALA-13885?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17938484#comment-17938484
 ] 

ASF subversion and git services commented on IMPALA-13885:
----------------------------------------------------------

Commit 8101b0784ccd1b87b85c1a055e92866614eb36a8 in impala's branch 
refs/heads/master from jasonmfehr
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=8101b0784 ]

IMPALA-13885: Add Query Id to Workload Management Insert Logs

When the workload management code inserts rows into the completed
queries table, it logs an entry with information on that insert but
does not include the query id in the log line. This lack of query id
causes extra steps to trace cases where the insert DML failed.

This change adds the query id to both the success and failure log
messages logged by the workload management main processing loop. It
also adds the error message to the failure log messages.

Additional minor cleanup was done to provide error messages on python
custom cluster test asserts, add additional asserts for the updated
log messages, and restore a trailing newline in the workload
management startup flags definition file.

All test_query_log.py tests passed locally. These tests were the only
tests that asserted the log messages that were modified.

Change-Id: I3c0816f9eb6bac8c891fd0e249de8863115bf466
Reviewed-on: http://gerrit.cloudera.org:8080/22656
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>


> Log Workload Management Insert DML Results
> ------------------------------------------
>
>                 Key: IMPALA-13885
>                 URL: https://issues.apache.org/jira/browse/IMPALA-13885
>             Project: IMPALA
>          Issue Type: Improvement
>            Reporter: Jason Fehr
>            Assignee: Jason Fehr
>            Priority: Critical
>
> When workload management executes its insert DML into the 
> sys.impala_query_log table, the INFO level log it writes needs to include the 
> query id.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to