You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "Minwoo Kang (Jira)" <ji...@apache.org> on 2020/03/12 16:21:00 UTC
[jira] [Comment Edited] (HBASE-23561) Look up of Region in Master
by encoded region name is O(n)
[ https://issues.apache.org/jira/browse/HBASE-23561?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17058060#comment-17058060 ]
Minwoo Kang edited comment on HBASE-23561 at 3/12/20, 4:20 PM:
---------------------------------------------------------------
For branch-2, I created a [pull request|[https://github.com/apache/hbase/pull/1281]].
For branch-1, it already looks up region name is O(1).
Therefore, I did not need to apply backport.
I close the issue.
was (Author: minwoo.kang):
[For branch-2|https://issues.apache.org/jira/browse/HBASE-23973], I created a [pull request|[https://github.com/apache/hbase/pull/1281]].
[For branch-1|https://issues.apache.org/jira/browse/HBASE-23972], it already looks up region name is O(1).
Therefore, I did not need to apply backport.
I close the issue.
> 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)