hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Andrew Purtell (JIRA)" <j...@apache.org>
Subject [jira] [Comment Edited] (HBASE-12972) Region, a supportable public/evolving subset of HRegion
Date Fri, 27 Feb 2015 04:58:04 GMT

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

Andrew Purtell edited comment on HBASE-12972 at 2/27/15 4:57 AM:
-----------------------------------------------------------------

That's reasonable. The reason the current patch doesn't do that is I'd like to avoid carrying
a ton of legacy around. Every  RegionObserver and RegionServerObserver and RegionServerServices
method that references the HRegion type will have to stay with deprecation and will get a
new twin. That's a lot of cruft. Nobody should be using HRegion methods anyway. Better to
make a clean break?


was (Author: apurtell):
That's reasonable. I'd like to avoid carrying legacy around. *Every* RegionObserver and RegionServerObserver
and RegionServerServices method that references the HRegion type will have to stay and will
get a new twin. That's a lot of cruft. Nobody should be using HRegion methods anyway. Better
to make a clean break? 

> Region, a supportable public/evolving subset of HRegion
> -------------------------------------------------------
>
>                 Key: HBASE-12972
>                 URL: https://issues.apache.org/jira/browse/HBASE-12972
>             Project: HBase
>          Issue Type: New Feature
>            Reporter: Andrew Purtell
>            Assignee: Andrew Purtell
>             Fix For: 2.0.0, 1.1.0
>
>         Attachments: HBASE-12972-0.98.patch, HBASE-12972-0.98.patch, HBASE-12972-0.98.patch,
HBASE-12972-0.98.patch
>
>
> On HBASE-12566, [~lhofhansl] proposed:
> {quote}
> Maybe we can have a {{Region}} interface that is to {{HRegion}} is what {{Store}} is
to {{HStore}}. Store marked with {{@InterfaceAudience.Private}} but used in some coprocessor
hooks.
> {quote}
> By example, now coprocessors have to reach into HRegion in order to participate in row
and region locking protocols, this is one area where the functionality is legitimate for coprocessors
but not for users, so an in-between interface make sense.
> In addition we should promote {{Store}}'s interface audience to LimitedPrivate(COPROC).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message