hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matteo Bertozzi (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-11757) Provide a common base abstract class for both RegionObserver and MasterObserver
Date Fri, 15 Aug 2014 20:45:19 GMT

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

Matteo Bertozzi commented on HBASE-11757:
-----------------------------------------

{quote}Copying default methods into a common abstract base is all I have in mind. At least
maintaining consistency among some duplicated code would fall on us rather than unfortunate
CP implementors.{quote}
ok, unless anyone else have a better idea I'm going with copying the code.
Do you prefer having the abstract class implementing both interfaces and copying both BaseMasterObserver
and BaseRegionObserver in, or extending for example BaseRegionObserver and implementing MasterObserver
like the visibility controller does?

> Provide a common base abstract class for both RegionObserver and MasterObserver
> -------------------------------------------------------------------------------
>
>                 Key: HBASE-11757
>                 URL: https://issues.apache.org/jira/browse/HBASE-11757
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Andrew Purtell
>             Fix For: 0.99.0, 2.0.0, 0.98.6
>
>
> Some security coprocessors extend both RegionObserver and MasterObserver, unfortunately
only one of the two can use the available base abstract class implementations. Provide a common
base abstract class for both the RegionObserver and MasterObserver interfaces. Update current
coprocessors that extend both interfaces to use the new common base abstract class.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message