Dan Burkert has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11566 )

Change subject: [hms] enable 'notifications.add.thrift.objects' in mini HMS
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11566/1/src/kudu/hms/hms_client.cc
File src/kudu/hms/hms_client.cc:

http://gerrit.cloudera.org:8080/#/c/11566/1/src/kudu/hms/hms_client.cc@185
PS1, Line 185:   // HMS notification log.
I think it's worth calling out that this is specifically for Cloudera's version 
of the HMS shipped in CDH.  Otherwise anyone reading this code that goes to the 
HMS codebase to find out more could be very confused.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I9f28f52bdddd59c5b0dc7120d62482de8becb1ee
Gerrit-Change-Number: 11566
Gerrit-PatchSet: 1
Gerrit-Owner: Hao Hao <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Tue, 02 Oct 2018 22:02:18 +0000
Gerrit-HasComments: Yes

Reply via email to