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

Bryan Beaudreault commented on HBASE-28346:
-------------------------------------------

Currently, checkQuota exists in RegionServerRpcQuotaManager. I don't think we 
should expose that whole class. Instead we can extract the checkQuota methods 
into a separate interface which RegionServerRpcQuotaManager implements, and 
just expose that interface on the RegionCoprocessorEnvironment.

> Expose checkQuota to Coprocessor Endpoints
> ------------------------------------------
>
>                 Key: HBASE-28346
>                 URL: https://issues.apache.org/jira/browse/HBASE-28346
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Bryan Beaudreault
>            Priority: Major
>
> Coprocessor endpoints may do non-trivial amounts of work, yet quotas do not 
> throttle them. We can't generically apply quotas to coprocessors because we 
> have no information on what a particular endpoint might do. One thing we 
> could do is expose checkQuota to the RegionCoprocessorEnvironment. This way, 
> coprocessor authors have the tools to ensure that quotas cover their 
> implementations.
> While adding this, we can update AggregationImplementation to call checkQuota 
> since those endpoints can be quite expensive.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to