You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "ASF subversion and git services (Jira)" <ji...@apache.org> on 2021/04/16 22:05:00 UTC

[jira] [Commented] (GEODE-8259) when client singlehop getAll encountered SerializationException, it should retry

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

ASF subversion and git services commented on GEODE-8259:
--------------------------------------------------------

Commit 4e841c21d834c1916299c75d56f47b35f0a74e2a in geode's branch refs/heads/GEODE-9064-JMX-filter-support_1.12-02 from Kirk Lund
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=4e841c2 ]

GEODE-8259: Fixup GetAllOpJUnitTest

Fixed Mockito compilation failure.


> when client singlehop getAll encountered SerializationException, it should retry
> --------------------------------------------------------------------------------
>
>                 Key: GEODE-8259
>                 URL: https://issues.apache.org/jira/browse/GEODE-8259
>             Project: Geode
>          Issue Type: Improvement
>            Reporter: Xiaojian Zhou
>            Assignee: Xiaojian Zhou
>            Priority: Major
>             Fix For: 1.12.1, 1.13.0, 1.14.0
>
>
> In GEOEDE-7090, DSFIDSerializerImpl.invokeFromData() will catch RunTimeException and throw. But convert Exception to be IOException. The idea is to avoid using SerializationException which is in another package. 
> However, if my fromData() failed with IndexOutOfBoundary exception (which is an RTE), it will be thrown directly instead of treating it in handleException(), thus my serialization exception will never get handled. 
> The fix is to merge the catch RunTimeException into catch Exception.  



--
This message was sent by Atlassian Jira
(v8.3.4#803005)