You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Hao Hao (Code Review)" <ge...@cloudera.org> on 2018/06/01 21:25:38 UTC

[kudu-CR] KUDU-2191 (12/n): Hive Metastore notification log event listener

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

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


Patch Set 14:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/8313/14/src/kudu/master/catalog_manager.cc
File src/kudu/master/catalog_manager.cc:

http://gerrit.cloudera.org:8080/#/c/8313/14/src/kudu/master/catalog_manager.cc@1549
PS14, Line 1549:     Status s = hms_catalog_->CreateTable(table->id(), req.name(), schema);
Maybe we should call WaitForCatchUp here as well?



-- 
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: 14
Gerrit-Owner: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <da...@apache.org>
Gerrit-Reviewer: Hao Hao <ha...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Comment-Date: Fri, 01 Jun 2018 21:25:38 +0000
Gerrit-HasComments: Yes