Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16388 )

Change subject: KUDU-3187: Enhance the HMS plugin to check if synchronization 
is enabled
......................................................................


Patch Set 8:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16388/8//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/16388/8//COMMIT_MSG@25
PS8, Line 25: the Kudu client runs in the plugin and communicates
            : with the cluster
> I didn't test this explicitly, but because I am using the credentials from
Good to know that HMS service takes care of that.

Do we know that it works as expected for Kudu client?  BTW, if it possible to 
add such a test as a part of MiniHms in ExternalMiniCluster?  There it will be 
easy to play with Kerberos-related settings using MiniKdc properties.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ib3588d72af1bb499202b47fca50a08876e13ea37
Gerrit-Change-Number: 16388
Gerrit-PatchSet: 8
Gerrit-Owner: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Greg Solovyev <gsolov...@cloudera.com>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 24 Sep 2020 19:33:54 +0000
Gerrit-HasComments: Yes

Reply via email to