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

stack commented on HBASE-17980:
-------------------------------

The reorder of RegionStates imports is my fault. I was adding the display stuff 
in here since it was from AM/RegionStates... I then went back on the change and 
called optimize on imports to undo the import changes. My fault.

I skimmed the patch. Looks good. Will look at it closer today. Can do move 
around in follow-up if I find any. The RegionInfoUtil looks good here given it 
only has the odd stuff in it.

[~brandboat] If you do not have time to bring this home, just say and I'll 
finish it up today... Thanks lads.

> Any HRegionInfo we give out should be immutable
> -----------------------------------------------
>
>                 Key: HBASE-17980
>                 URL: https://issues.apache.org/jira/browse/HBASE-17980
>             Project: HBase
>          Issue Type: Sub-task
>    Affects Versions: 2.0.0
>            Reporter: Chia-Ping Tsai
>            Assignee: Kuan-Po Tseng
>              Labels: beginner
>             Fix For: 2.0.0-alpha-3
>
>         Attachments: HBASE-17980.master.001.patch, 
> HBASE-17980.master.002.patch, HBASE-17980.master.v0.patch, 
> HBASE-17980.master.v1.patch, HBASE-17980-master.v2.patch, 
> HBASE-17980-master.v2.patch, HBASE-17980.master.v3.patch, 
> HBASE-17980.master.v4.patch
>
>
> This is similar to HBASE-15583.
> # Introduce RegionInfo class. HRegionInfo will extend RegionInfo.
> # Deprecate HRegionInfo to be removed in 3.0
> # RegionInfo contain all of the read-only methods of HRegionInfo
> # Add "RegionInfo Builder"



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to