Vincent Tran has posted comments on this change. (
http://gerrit.cloudera.org:8080/7241 )
Change subject: IMPALA-2782: Allow impala-shell to connect directly to impalad
when configured with load balancer and kerberos.
......................................................................
Patch Set 2:
> Patch Set 2:
>
> I think we should use impalad[1] for the port still since then you'll still
> get the default 21000 if you specify nothing and its less you would have to
> enter for the -b flag. I also question whether this should be called
> something like kerberos_host_name so its more inline with
> kerberos_service_name change.
>
> host, port = self.lb.encode('ascii', 'ignore'), int(self.impalad[1])
Good point.
--
To view, visit http://gerrit.cloudera.org:8080/7241
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I4726226a7a3817421b133f74dd4f4cf8c52135f9
Gerrit-Change-Number: 7241
Gerrit-PatchSet: 2
Gerrit-Owner: Vincent Tran <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Philip Zeyliger <[email protected]>
Gerrit-Reviewer: Vincent Tran <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Comment-Date: Sun, 11 Feb 2018 03:04:30 +0000
Gerrit-HasComments: No
