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

Change subject: IMPALA-6477: rpc-mgr-kerberized-test fails on CentOS 6.4
......................................................................


Patch Set 1:

(2 comments)

> (2 comments)
 >
 > So, thrift-server-test didn't need this workaround because it was
 > using "impala/localhost" instead of "impala/127.0.0.1", right ?

Yes, that's right. We use a non-numeric principal for thrift-server-test, so 
the bug isn't exposed.

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

http://gerrit.cloudera.org:8080/#/c/9519/1/be/src/testutil/mini-kdc-wrapper.cc@69
PS1, Line 69:      // Enable the workaround for MIT krb5 1.10 bugs from 
krb5_realm_override.cc.
> nit: indent off
Done


http://gerrit.cloudera.org:8080/#/c/9519/1/be/src/testutil/mini-kdc-wrapper.cc@70
PS1, Line 70: "true"
> My understanding is that the workaround will be in effect as long as this e
Yes, that's right



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I7227551aabd1ef4f8e8608fefb74293f9f763e13
Gerrit-Change-Number: 9519
Gerrit-PatchSet: 1
Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com>
Gerrit-Reviewer: Michael Ho <k...@cloudera.com>
Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com>
Gerrit-Comment-Date: Wed, 07 Mar 2018 00:09:45 +0000
Gerrit-HasComments: Yes

Reply via email to