[ https://issues.apache.org/jira/browse/HBASE-21025?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16575350#comment-16575350 ]
stack commented on HBASE-21025: ------------------------------- Looks good. For the cache, if a table is removed, we for sure clear its entry in the cache? Should the cache be soft reference? If making new patch, change name from private final ConcurrentMap<TableName, TableState.State> tn2State = new ConcurrentHashMap<>(); to tableName2State. Yeah, this is important change. We need it bad. > Add cache for TableStateManager > ------------------------------- > > Key: HBASE-21025 > URL: https://issues.apache.org/jira/browse/HBASE-21025 > Project: HBase > Issue Type: Sub-task > Reporter: Duo Zhang > Assignee: Duo Zhang > Priority: Major > Fix For: 3.0.0, 2.2.0, 2.1.1 > > Attachments: HBASE-21025-v1.patch, HBASE-21025.patch > > > After HBASE-20881, we will check whether a table is disabled in SCP, so we > need to add cache for it to improve MTTR, and also reduce the request to meta. -- This message was sent by Atlassian JIRA (v7.6.3#76005)