hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Anoop Sam John (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-18298) RegionServerServices Interface cleanup for CP expose
Date Fri, 22 Sep 2017 10:19:00 GMT

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

Anoop Sam John commented on HBASE-18298:

bq.Yeah, name CoprocessorRSServices as CompressorRegionServerServices and method name should
be getCompressorRegionServerServices
U mean Coprocessor  not Compressor right?
bq.Or could we insert an immutable OnlineRegions? Or make a subinterface that adds the editing
I thought of making 2 interfaces with the base for the CP expose and the sub with adding the
APIs to update the online regions..  Thought of going with smaller change :-)..  Ya will do..
I think the 2 interface way is better than we have the API in the CP exposed and making that
to throw Exception.  Then every getter  from the CP exposed API have to make a wrapper of
this object and make the wrapper impl to throw Exception on the add/remove APIs. I prefer
the simpler way of 2 interfaces.

> RegionServerServices Interface cleanup for CP expose
> ----------------------------------------------------
>                 Key: HBASE-18298
>                 URL: https://issues.apache.org/jira/browse/HBASE-18298
>             Project: HBase
>          Issue Type: Sub-task
>          Components: Coprocessors
>            Reporter: Anoop Sam John
>            Assignee: Anoop Sam John
>            Priority: Critical
>             Fix For: 2.0.0-alpha-4
>         Attachments: HBASE-18298.patch, HBASE-18298_V2.patch, HBASE-18298_V3.patch, HBASE-18298_V4.patch

This message was sent by Atlassian JIRA

View raw message