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

Change subject: KUDU-2191 (10/n): Hive Metastore notification log event listener
......................................................................


Patch Set 7:

> I can see how a double apply could lead to errors (i.e. trying to add the 
> same > column twice)

Another thing to note about this question: Kudu _will not_ be responding to ADD 
COLUMN or DROP COLUMN alterations by modifying the Kudu catalog.  We are simply 
going to respond to these notifications from the HMS, and overwrite the HMS's 
version of the table metadata with our own.  The _only_ notifications from the 
HMS which result in a modification of the Kudu catalog are DROP TABLE and ALTER 
TABLE RENAME.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I32ed099c44a593ffe514152135957018f21ed775
Gerrit-Change-Number: 8313
Gerrit-PatchSet: 7
Gerrit-Owner: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Comment-Date: Thu, 12 Apr 2018 19:27:33 +0000
Gerrit-HasComments: No

Reply via email to