hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "stack (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-17980) Any HRegionInfo we give out should be immutable
Date Wed, 13 Sep 2017 14:49:00 GMT

    [ 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)

Mime
View raw message