[jira] [Commented] (HBASE-23561) Look up of Region in Master by encoded region name is O(n)

["Michael Stack (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/HBASE-23561?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17056236#comment-17056236 ] 

Michael Stack commented on HBASE-23561:
---------------------------------------

[~minwoo.kang] The new PR looks like an addendum tying the two Maps together? Its looks like it is critical and needs to be applied, is that right?

> Look up of Region in Master by encoded region name is O(n)
> ----------------------------------------------------------
>
>                 Key: HBASE-23561
>                 URL: https://issues.apache.org/jira/browse/HBASE-23561
>             Project: HBase
>          Issue Type: Bug
>    Affects Versions: 3.0.0
>            Reporter: Michael Stack
>            Assignee: Minwoo Kang
>            Priority: Trivial
>             Fix For: 3.0.0
>
>
> {{  public RegionState getRegionState(final String encodedRegionName) {
>     // TODO: Need a map <encodedName, ...> but it is just dispatch merge...
>     for (RegionStateNode node: regionsMap.values()) {
>       if (node.getRegionInfo().getEncodedName().equals(encodedRegionName)) {
>         return node.toRegionState();
>       }
>     }
>     return null;
>   }}}
> It is not used much so making it trivial.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)