[
https://issues.apache.org/jira/browse/PHOENIX-4683?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16423293#comment-16423293
]
James Taylor commented on PHOENIX-4683:
---------------------------------------
LGTM, but any reason not to create just one
DelegateRegionCoprocessorEnvironment?
> Cap timeouts for stats precompact hook logic
> --------------------------------------------
>
> Key: PHOENIX-4683
> URL: https://issues.apache.org/jira/browse/PHOENIX-4683
> Project: Phoenix
> Issue Type: Bug
> Affects Versions: 4.13.0
> Reporter: Vincent Poon
> Assignee: Vincent Poon
> Priority: Major
> Attachments: PHOENIX-4683.v1.0.98.patch, PHOENIX-4683.v2.0.98.patch
>
>
> In UngroupedAggregateRegionObserver#preCompact we call
> DefaultStatisticsCollector.createCompactionScanner. It uses the env config
> which in turn contains the RS server rpc timeout of 20 minutes. That's too
> long for a compaction hook.
> Like in PHOENIX-4169, we should cap the timeout so the compaction doesn't get
> blocked.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
