phoenix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hudson (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (PHOENIX-3938) Don't throw IOException out of coprocessor hooks
Date Thu, 06 Jul 2017 22:51:00 GMT

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

Hudson commented on PHOENIX-3938:
---------------------------------

FAILURE: Integrated in Jenkins build Phoenix-master #1673 (See [https://builds.apache.org/job/Phoenix-master/1673/])
PHOENIX-3938 Don't throw IOException out of coprocessor hooks (samarth: rev 77509422266adfc190892bd57faf8b81b072519e)
* (edit) phoenix-core/src/main/java/org/apache/phoenix/coprocessor/ServerCachingEndpointImpl.java
* (edit) phoenix-core/src/main/java/org/apache/phoenix/coprocessor/MetaDataEndpointImpl.java
* (edit) phoenix-core/src/main/java/org/apache/phoenix/coprocessor/UngroupedAggregateRegionObserver.java


> 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
>             Fix For: 4.12.0, 4.11.1
>
>         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)

Mime
View raw message