You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by he...@apache.org on 2017/06/14 21:49:09 UTC
[1/2] incubator-impala git commit: IMPALA-5492: Fix incorrect newline
character in the LDAP message
Repository: incubator-impala
Updated Branches:
refs/heads/master 11ec9f195 -> e2532a96c
IMPALA-5492: Fix incorrect newline character in the LDAP message
The introduction has redundant '\' in impala-shell when using LDAP.
I fix this issue by removing extraneous '\' in introduction when
impala-shell using LDAP.
Change-Id: I30c601ab255a4882260f7be23b5763ef8ec76d28
Reviewed-on: http://gerrit.cloudera.org:8080/7166
Reviewed-by: Bharath Vissapragada <bh...@cloudera.com>
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/9ef5ad48
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/9ef5ad48
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/9ef5ad48
Branch: refs/heads/master
Commit: 9ef5ad48a878f77fd78965d9d97920cfae88bd5f
Parents: 11ec9f1
Author: davidxdh <xu...@zte.com.cn>
Authored: Tue Jun 13 11:21:25 2017 +0800
Committer: Impala Public Jenkins <im...@gerrit.cloudera.org>
Committed: Wed Jun 14 08:26:29 2017 +0000
----------------------------------------------------------------------
shell/impala_shell.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/9ef5ad48/shell/impala_shell.py
----------------------------------------------------------------------
diff --git a/shell/impala_shell.py b/shell/impala_shell.py
index d3495f7..d143ef1 100755
--- a/shell/impala_shell.py
+++ b/shell/impala_shell.py
@@ -1388,7 +1388,7 @@ if __name__ == "__main__":
intro = WELCOME_STRING
if not options.ssl and options.creds_ok_in_clear and options.use_ldap:
- intro += ("\n\\nLDAP authentication is enabled, but the connection to Impala is " +
+ intro += ("\n\nLDAP authentication is enabled, but the connection to Impala is "
"not secured by TLS.\nALL PASSWORDS WILL BE SENT IN THE CLEAR TO IMPALA.\n")
shell = ImpalaShell(options)
[2/2] incubator-impala git commit: IMPALA-5506: Add stdin description
to help information of query_file option
Posted by he...@apache.org.
IMPALA-5506: Add stdin description to help information of query_file option
Help information of query_file option in impala-shell misses stdin
description.
I fix this issue by adding stdin description to help information of
query_file option.
Change-Id: I0264174fd062497c72891b31cf9ac1ba6c00f716
Reviewed-on: http://gerrit.cloudera.org:8080/7178
Reviewed-by: Henry Robinson <he...@cloudera.com>
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/e2532a96
Tree: http://git-wip-us.apache.org/repos/asf/incubator-impala/tree/e2532a96
Diff: http://git-wip-us.apache.org/repos/asf/incubator-impala/diff/e2532a96
Branch: refs/heads/master
Commit: e2532a96c81ecfa2dc763306e96eb340fb49afe3
Parents: 9ef5ad4
Author: davidxdh <xu...@zte.com.cn>
Authored: Wed Jun 14 13:53:54 2017 +0800
Committer: Impala Public Jenkins <im...@gerrit.cloudera.org>
Committed: Wed Jun 14 20:51:18 2017 +0000
----------------------------------------------------------------------
shell/option_parser.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-impala/blob/e2532a96/shell/option_parser.py
----------------------------------------------------------------------
diff --git a/shell/option_parser.py b/shell/option_parser.py
index 0a30ed4..6092439 100755
--- a/shell/option_parser.py
+++ b/shell/option_parser.py
@@ -84,7 +84,8 @@ def get_option_parser(defaults):
parser.add_option("-q", "--query", dest="query",
help="Execute a query without the shell")
parser.add_option("-f", "--query_file", dest="query_file",
- help="Execute the queries in the query file, delimited by ;")
+ help="Execute the queries in the query file, delimited by ;."
+ " Queries may be read from stdin if the argument to -f is -.")
parser.add_option("-k", "--kerberos", dest="use_kerberos",
action="store_true", help="Connect to a kerberized impalad")
parser.add_option("-o", "--output_file", dest="output_file",