[
https://issues.apache.org/jira/browse/HBASE-15843?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15290525#comment-15290525
]
Hudson commented on HBASE-15843:
--------------------------------
SUCCESS: Integrated in HBase-1.4 #162 (See
[https://builds.apache.org/job/HBase-1.4/162/])
HBASE-15843 Replace RegionState.getRegionInTransition() Map with a Set
(matteo.bertozzi: rev e5eea96ca6c55f285f0fd8fb0b19c7d161650546)
* hbase-server/src/main/java/org/apache/hadoop/hbase/master/RegionStates.java
*
hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestSplitTransactionOnCluster.java
*
hbase-server/src/test/java/org/apache/hadoop/hbase/coprocessor/TestMasterObserver.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/TestDrainingServer.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/master/HMaster.java
*
hbase-server/src/main/java/org/apache/hadoop/hbase/master/MasterDumpServlet.java
*
hbase-server/src/main/java/org/apache/hadoop/hbase/master/AssignmentManager.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsckRepair.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/util/TestHBaseFsck.java
* hbase-server/src/main/java/org/apache/hadoop/hbase/util/HBaseFsck.java
* hbase-shell/src/main/ruby/hbase/admin.rb
* hbase-client/src/main/java/org/apache/hadoop/hbase/protobuf/ProtobufUtil.java
* hbase-server/src/test/java/org/apache/hadoop/hbase/master/Mocking.java
*
hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/AssignmentManagerStatusTmpl.jamon
*
hbase-server/src/test/java/org/apache/hadoop/hbase/master/TestMasterStatusServlet.java
* hbase-client/src/main/java/org/apache/hadoop/hbase/ClusterStatus.java
> 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, 1.4.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)
