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

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

.003 is based on v6. It does below:

* Fixes broke tests by changing RegionInfoBuilder so it does NOT allow you 
build the hbase:meta FIRST_META_REGION; you must use the static define 
everywhere. It took a bit of time to track down 'everywhere'. Hopefully I got 
them all.
* I killed RegionInfoUtil again. It was just awkward. Moved its static utility 
back into RegionInfo which is where you'd go looking for it anyways. Marked 
near all with Audience Private because most of these methods are awful hacks 
and should go away.
* Just moved RegionInfoDisplay to client package beside RegionInfo and 
RegionInfoBuilder. Its not utility. Its wonky stuff needed in UI.

Otherwise it is all v6. [~brandboat] and [~chia7712] see what you think. I put 
it up on rb. See link at top of this issue. Thanks.

> 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.003.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, 
> HBASE-17980.master.v5.patch, HBASE-17980.master.v6.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