You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by to...@apache.org on 2016/03/01 08:46:18 UTC
[2/4] incubator-kudu git commit: sys_catalog: put the correct tablet
ID in delete catalog table requests
sys_catalog: put the correct tablet ID in delete catalog table requests
I don't think this has any real effect because we only ever delete the
catalog table in unit tests. And even if those tests used multiple masters,
I can't see where a follower would use the tablet ID from the WriteRequestPB
and not from the ConsensusRequestPB.
Change-Id: I5a3f56fea5f6f590f1bc76c32fe0a44cffe9cfcc
Reviewed-on: http://gerrit.cloudera.org:8080/2369
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon <to...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/incubator-kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-kudu/commit/fdfa218c
Tree: http://git-wip-us.apache.org/repos/asf/incubator-kudu/tree/fdfa218c
Diff: http://git-wip-us.apache.org/repos/asf/incubator-kudu/diff/fdfa218c
Branch: refs/heads/master
Commit: fdfa218c37e3dc0163daea2108d556d7e6697c99
Parents: ffbe5a6
Author: Adar Dembo <ad...@cloudera.com>
Authored: Mon Feb 29 20:34:09 2016 -0800
Committer: Adar Dembo <ad...@cloudera.com>
Committed: Tue Mar 1 07:36:37 2016 +0000
----------------------------------------------------------------------
src/kudu/master/sys_catalog.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/incubator-kudu/blob/fdfa218c/src/kudu/master/sys_catalog.cc
----------------------------------------------------------------------
diff --git a/src/kudu/master/sys_catalog.cc b/src/kudu/master/sys_catalog.cc
index c443ea6..f25e66a 100644
--- a/src/kudu/master/sys_catalog.cc
+++ b/src/kudu/master/sys_catalog.cc
@@ -414,7 +414,7 @@ Status SysCatalogTable::DeleteTable(const TableInfo *table) {
"table", table->ToString());
WriteRequestPB req;
WriteResponsePB resp;
- req.set_tablet_id(kSysCatalogTableColMetadata);
+ req.set_tablet_id(kSysCatalogTabletId);
RETURN_NOT_OK(SchemaToPB(schema_, req.mutable_schema()));
KuduPartialRow row(&schema_);