[ 
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)

Reply via email to