Sailesh Mukil has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8270 )

Change subject: IMPALA-5053: [SECURITY] Make KRPC work with Kerberos
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/8270/1/be/src/testutil/mini-kdc-wrapper.cc
File be/src/testutil/mini-kdc-wrapper.cc:

http://gerrit.cloudera.org:8080/#/c/8270/1/be/src/testutil/mini-kdc-wrapper.cc@17
PS1, Line 17:
I should have done this in the kinit patch, but since this file needs to be 
reused by the rpc-mgr-test, I had to move it to a new file.

I'm not changing the kinit patch, since that's already close to review 
completion, and this is mostly a mechanical move to a new file.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8cec5cca5fdb4b1d46bab19e86cb1a8a3ad718fd
Gerrit-Change-Number: 8270
Gerrit-PatchSet: 1
Gerrit-Owner: Sailesh Mukil <[email protected]>
Gerrit-Reviewer: Sailesh Mukil <[email protected]>
Gerrit-Comment-Date: Fri, 13 Oct 2017 05:40:21 +0000
Gerrit-HasComments: Yes

Reply via email to