Mailing-List: contact dev-help@hbase.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@hbase.apache.org
Date: Tue, 17 May 2016 02:55:12 +0000 (UTC)
From: "Matteo Bertozzi (JIRA)" <jira@apache.org>
To: dev@hbase.apache.org
Message-ID: <JIRA.12970246.1463453711000.205223.1463453712880@Atlassian.JIRA>
In-Reply-To: <JIRA.12970246.1463453711000@Atlassian.JIRA>
References: <JIRA.12970246.1463453711000@Atlassian.JIRA> <JIRA.12970246.1463453711788@arcas>
Subject: [jira] [Created] (HBASE-15843) Replace
 RegionState.getRegionInTransition() Map with a Set
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Tue, 17 May 2016 02:55:14 -0000

Matteo Bertozzi created HBASE-15843:
---------------------------------------

             Summary: 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: 1.2.1, 2.0.0, 1.3.0
            Reporter: Matteo Bertozzi
            Assignee: Matteo Bertozzi
            Priority: Trivial
             Fix For: 2.0.0


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)