Bharath Vissapragada has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/9506 )

Change subject: IMPALA-6610: Impala shell fetches the value of 
ldap_password_cmd incorrectly
......................................................................


Patch Set 4:

Maybe we could fix (a) or (c) separately (or add more documentation if you 
think that is lacking) but my opinion is that the onus is on the end user to 
provide the correct command and the shell shouldn't make any changes like 
stripping newlines etc.

Wondering if others have a different opinion on this?


--
To view, visit http://gerrit.cloudera.org:8080/9506
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie570166aea62af223905b7f0124e9efb15a88ac7
Gerrit-Change-Number: 9506
Gerrit-PatchSet: 4
Gerrit-Owner: Donghui Xu <xu.dong...@zte.com.cn>
Gerrit-Reviewer: Bharath Vissapragada <bhara...@cloudera.com>
Gerrit-Reviewer: Donghui Xu <xu.dong...@zte.com.cn>
Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com>
Gerrit-Comment-Date: Fri, 09 Mar 2018 20:03:15 +0000
Gerrit-HasComments: No

Reply via email to