impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Donghui Xu (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-5514: Add the joint judgment of ldap password cmd and ldap
Date Thu, 15 Jun 2017 09:38:05 GMT
Donghui Xu has uploaded a new patch set (#2).

Change subject: IMPALA-5514: Add the joint judgment of ldap_password_cmd and ldap
......................................................................

IMPALA-5514: Add the joint judgment of ldap_password_cmd and ldap

When only with ldap_password_cmd option, impala-shell runs successfully.

I solved this problem by adding the joint judgment of ldap_password_cmd 
and ldap, that is, ldap_password_cmd option will only take effect if 
ldap option presents.

Change-Id: I3711d8a0eca2fa8612e2943fa9121945db6b012e
---
M shell/impala_shell.py
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/88/7188/2
-- 
To view, visit http://gerrit.cloudera.org:8080/7188
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3711d8a0eca2fa8612e2943fa9121945db6b012e
Gerrit-PatchSet: 2
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Donghui Xu <xu.donghui@zte.com.cn>

Mime
View raw message