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 <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Greg Solovyev <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Thu, 24 Sep 2020 19:33:54 +0000
Gerrit-HasComments: Yes

Reply via email to