You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "wangning (Code Review)" <ge...@cloudera.org> on 2020/05/28 11:32:42 UTC

[kudu-CR] KUDU-3097 whether master load deleted table info could be configurable

Hello Tidy Bot, Kudu Jenkins, Adar Lieber-Dembo, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/15616

to look at the new patch set (#12).

Change subject: KUDU-3097 whether master load deleted table info could be configurable
......................................................................

KUDU-3097 whether master load deleted table info could be configurable

When deleting table, the records in master tablet are just marked as
deleted, and all the table structure of table and tablet id are kept in the master
tablet forever.

From the perspective of storage, it's not big deal.
But it result in heavy burden if too many tables were deleted in past time.
By the way, msater only have one tablet and only leader works in
cluster, it would lead to imbalance memory usage.

This commit make it configurable to load those things into memory.

Change-Id: I1088080d706edadeed9d93f54704c7f4bfe2b6a2
---
M src/kudu/master/catalog_manager.cc
1 file changed, 17 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/16/15616/12
-- 
To view, visit http://gerrit.cloudera.org:8080/15616
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1088080d706edadeed9d93f54704c7f4bfe2b6a2
Gerrit-Change-Number: 15616
Gerrit-PatchSet: 12
Gerrit-Owner: wangning <19...@gmail.com>
Gerrit-Reviewer: Adar Lieber-Dembo <ad...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: wangning <19...@gmail.com>