You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@impala.apache.org by "Sailesh Mukil (Code Review)" <ge...@cloudera.org> on 2017/09/01 03:09:12 UTC

[Impala-ASF-CR] security: only lookup hostname if HOST substitution is required

Sailesh Mukil has posted comments on this change.

Change subject: security: only lookup hostname if _HOST substitution is required
......................................................................


Patch Set 4: Verified+1

Ran the GVO that succeeded of 4 commits together from the top level patch:
https://gerrit.cloudera.org/#/c/7896/

GVO link: https://jenkins.impala.io/job/gerrit-verify-dryrun/1187/

Verifying and submitting manually

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I5de8647d6cf63ea70d880fa530fa289e8bae24fe
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Sailesh Mukil <sa...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Michael Ho <kw...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sa...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-HasComments: No