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

Kuan-Po Tseng commented on HBASE-17980:
---------------------------------------

[~stack] Almost done. My last question is could RegionInfoDisplay.java be 
placed in hbase-client module (in org.apache.hadoop.hbase.util) ? So we could 
replace all these deprecated methods below in HRegionInfo.

{code:java}
  public static String getDescriptiveNameFromRegionStateForDisplay(RegionState 
state, Configuration conf)
  public static byte[] getEndKeyForDisplay(RegionInfo ri, Configuration conf)
  public static byte[] getStartKeyForDisplay(RegionInfo ri, Configuration conf) 
  public static String getRegionNameAsStringForDisplay(RegionInfo ri, 
Configuration conf)
  public static byte[] getRegionNameForDisplay(RegionInfo ri, Configuration 
conf)
{code}

> 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