Repository: incubator-impala
Updated Branches:
  refs/heads/master be8d15122 -> 676a88f34


IMPALA-4293: query profile should include error log

This puts the string-ified error log into an "Errors:"
field in the profile.

Testing:
Visually checked that the field showed up in the profiles in
the web UI. Ran core tests.

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


Project: http://git-wip-us.apache.org/repos/asf/incubator-impala/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-impala/commit/cfcb5f40
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/cfcb5f40
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/cfcb5f40

Branch: refs/heads/master
Commit: cfcb5f40017bc596bb26d4736ca73a318f5e0e6d
Parents: be8d151
Author: Tim Armstrong <[email protected]>
Authored: Tue Apr 4 10:43:22 2017 -0700
Committer: Impala Public Jenkins <[email protected]>
Committed: Wed Apr 5 03:03:57 2017 +0000

----------------------------------------------------------------------
 be/src/service/impala-server.cc | 2 ++
 1 file changed, 2 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/cfcb5f40/be/src/service/impala-server.cc
----------------------------------------------------------------------
diff --git a/be/src/service/impala-server.cc b/be/src/service/impala-server.cc
index 313708e..80c1507 100644
--- a/be/src/service/impala-server.cc
+++ b/be/src/service/impala-server.cc
@@ -966,6 +966,8 @@ Status ImpalaServer::UnregisterQuery(const TUniqueId& 
query_id, bool check_infli
       exec_summary = PrintExecSummary(summary);
     }
     exec_state->summary_profile()->AddInfoString("ExecSummary", exec_summary);
+    exec_state->summary_profile()->AddInfoString("Errors",
+        exec_state->coord()->GetErrorLog());
 
     const unordered_set<TNetworkAddress>& unique_hosts =
         exec_state->schedule()->unique_hosts();

Reply via email to