[
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 methods?
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
(v6.4.14#64029)
