You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by ar...@apache.org on 2018/05/08 23:55:11 UTC

[1/2] impala git commit: IMPALA-6993: Don't log stack trace in Coordinator::BackendState::Exec()

Repository: impala
Updated Branches:
  refs/heads/master 96c9dac28 -> f976318cc


IMPALA-6993: Don't log stack trace in Coordinator::BackendState::Exec()

We should use Status::Expected() here, just like it's used
above. The stack trace isn't interesting and the error is expected
once we get down this path. (I'm not sure why we don't just use
exec_status though, but presumably the prefix was added for a
reason).

Change-Id: I9cd015ce0a5af51de73d591f07c01a80cda942e0
Reviewed-on: http://gerrit.cloudera.org:8080/10348
Reviewed-by: Tim Armstrong <ta...@cloudera.com>
Tested-by: Impala Public Jenkins <im...@cloudera.com>


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

Branch: refs/heads/master
Commit: df2798ea977e1cca87842c524b35ac46a51d5c28
Parents: 96c9dac
Author: Dan Hecht <dh...@cloudera.com>
Authored: Tue May 8 12:13:40 2018 -0700
Committer: Impala Public Jenkins <im...@cloudera.com>
Committed: Tue May 8 23:40:24 2018 +0000

----------------------------------------------------------------------
 be/src/runtime/coordinator-backend-state.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/impala/blob/df2798ea/be/src/runtime/coordinator-backend-state.cc
----------------------------------------------------------------------
diff --git a/be/src/runtime/coordinator-backend-state.cc b/be/src/runtime/coordinator-backend-state.cc
index 303a5a3..3099412 100644
--- a/be/src/runtime/coordinator-backend-state.cc
+++ b/be/src/runtime/coordinator-backend-state.cc
@@ -190,7 +190,7 @@ void Coordinator::BackendState::Exec(
     const string& err_msg = Substitute(ERR_TEMPLATE, PrintId(query_id_),
         exec_status.msg().GetFullMessageDetails());
     VLOG_QUERY << err_msg;
-    status_ = Status(err_msg);
+    status_ = Status::Expected(err_msg);
     return;
   }
 


[2/2] impala git commit: [DOCS] Fixed a typo in CREATE TABLE syntax

Posted by ar...@apache.org.
[DOCS] Fixed a typo in CREATE TABLE syntax

Change-Id: I968b92dc10d0426fc6295965f909e91b78379522
Reviewed-on: http://gerrit.cloudera.org:8080/10350
Reviewed-by: Alex Rodoni <ar...@cloudera.com>
Tested-by: Impala Public Jenkins <im...@cloudera.com>


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

Branch: refs/heads/master
Commit: f976318cc1a7e1e9779e5c7695033536c8c22593
Parents: df2798e
Author: Alex Rodoni <ar...@cloudera.com>
Authored: Tue May 8 16:33:34 2018 -0700
Committer: Impala Public Jenkins <im...@cloudera.com>
Committed: Tue May 8 23:42:11 2018 +0000

----------------------------------------------------------------------
 docs/topics/impala_create_table.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/impala/blob/f976318c/docs/topics/impala_create_table.xml
----------------------------------------------------------------------
diff --git a/docs/topics/impala_create_table.xml b/docs/topics/impala_create_table.xml
index 415b7be..0af25b0 100644
--- a/docs/topics/impala_create_table.xml
+++ b/docs/topics/impala_create_table.xml
@@ -96,7 +96,7 @@ under the License.
   [WITH SERDEPROPERTIES ('<varname>key1</varname>'='<varname>value1</varname>', '<varname>key2</varname>'='<varname>value2</varname>', ...)]
   [STORED AS <varname>file_format</varname>]
   [LOCATION '<varname>hdfs_path</varname>']
-<ph rev="1.4.0">  CACHED IN '<varname>pool_name</varname>'</ph> <ph rev="2.2.0">[WITH REPLICATION = <varname>integer</varname>]</ph> | UNCACHED]
+  <ph rev="1.4.0">[CACHED IN '<varname>pool_name</varname>'</ph> <ph rev="2.2.0">[WITH REPLICATION = <varname>integer</varname>]</ph> | UNCACHED]
   [TBLPROPERTIES ('<varname>key1</varname>'='<varname>value1</varname>', '<varname>key2</varname>'='<varname>value2</varname>', ...)]
 </codeblock>