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

["ASF subversion and git services (Jira)" <ji...@apache.org>]

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

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

Commit 16f2093fa59b9f0f3b7600cc365d922d78d6e8dd in geode's branch refs/heads/support/1.12 from Xiaojian Zhou
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=16f2093 ]

GEODE-8259: when client singlehop getAll encountered SerializationException, it should retry (#5253)

    Co-authored-by: Xiaojian Zhou <gz...@pivotal.io>
    Co-authored-by: Anil <ag...@pivotal.io>

(cherry picked from commit ee9a4b05277ff531d0d89d5d0fb65f63063557e3)


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