[
https://issues.apache.org/jira/browse/HBASE-15843?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Matteo Bertozzi updated HBASE-15843:
------------------------------------
Attachment: HBASE-15843-v2.patch
HBASE-15843-v2_branch-1.patch
attaching patch for branch-1 too, since this will cleanup a bunch of things and
I found broken stuff like contains() with a tableName instead of the
encodedName. (the compiler will not give errors for the wrong type because
contains() and containsKey() have Object as arg)
> Replace RegionState.getRegionInTransition() Map with a Set
> ----------------------------------------------------------
>
> Key: HBASE-15843
> URL: https://issues.apache.org/jira/browse/HBASE-15843
> Project: HBase
> Issue Type: Improvement
> Components: master, Region Assignment
> Affects Versions: 2.0.0, 1.3.0, 1.2.1
> Reporter: Matteo Bertozzi
> Assignee: Matteo Bertozzi
> Priority: Trivial
> Fix For: 2.0.0
>
> Attachments: HBASE-15843-v0.patch, HBASE-15843-v1.patch,
> HBASE-15843-v2.patch, HBASE-15843-v2_branch-1.patch
>
>
> RegionState.getRegionInTransition() is always used as a Set.
> replace the Map with a Set, avoid some allocation and extra code.
> also ClusterStatus.RegionInTransition has duplicated information.
> The spec field contains the regionName (not encoded).
> but we have the same info as part of the region_state with the HRegionInfo
> serialized.
> unfortunately I don't think we can get rid of 'spec' that being a required
> field.
> {noformat}
> message RegionInTransition {
> required RegionSpecifier spec = 1;
> required RegionState region_state = 2;
> }
> {noformat}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
