Todd Lipcon has submitted this change and it was merged.

Change subject: KUDU-745 / KUDU-1463 tablet's table id attribute is empty string
......................................................................


KUDU-745 / KUDU-1463 tablet's table id attribute is empty string

This fixes the issue where the table_id attribute of a tablet
in /metrics was an empty string.

As part of this patch, I also ended up picking a couple of magic
table_id's. The significant one is that the table_id of sys.catalog
is "sys.catalog.id".

Change-Id: I1795a176312f6c2a55c34b7b684ee408f1cf8732
Reviewed-on: http://gerrit.cloudera.org:8080/3183
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon <t...@apache.org>
---
M src/kudu/consensus/log-test-base.h
M src/kudu/master/sys_catalog.cc
M src/kudu/master/sys_catalog.h
M src/kudu/tablet/tablet-harness.h
M src/kudu/tablet/tablet_bootstrap-test.cc
M src/kudu/tablet/tablet_metadata.cc
M src/kudu/tablet/tablet_metadata.h
M src/kudu/tserver/remote_bootstrap_client.cc
M src/kudu/tserver/tablet_server-test.cc
M src/kudu/tserver/ts_tablet_manager.cc
10 files changed, 20 insertions(+), 5 deletions(-)

Approvals:
  Todd Lipcon: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1795a176312f6c2a55c34b7b684ee408f1cf8732
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Will Berkeley <wdberke...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>

Reply via email to