[
https://issues.apache.org/jira/browse/PHOENIX-3938?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Samarth Jain updated PHOENIX-3938:
----------------------------------
Attachment: PHOENIX-3938_nowhitespacediff.patch
No whitespace diff patch. [~jamestaylor] or [~tdsilva], please review.
I am not 100% sure about how to handle InterruptedException in various hooks in
UngroupedAggregateRegionObserver. For now, I followed the same pattern as
before to throw it back as an IOException but I am not sure. [~rajeshbabu] or
[~enis] - do you have any insights on how should these InterruptedExceptions be
handled in UngroupedAggregateRegionObserver?
> Don't throw IOException out of coprocessor hooks
> ------------------------------------------------
>
> Key: PHOENIX-3938
> URL: https://issues.apache.org/jira/browse/PHOENIX-3938
> Project: Phoenix
> Issue Type: Bug
> Reporter: James Taylor
> Assignee: Samarth Jain
> Attachments: PHOENIX-3938_nowhitespacediff.patch
>
>
> We shouldn't wrap and throw exceptions as an IOException from coprocessor
> hooks since that'll trigger the retry logic which just delays throwing the
> exception back to the client. Instead, we should wrap in a
> DoNotRetryIOException.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)