[ 
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)

Reply via email to