You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "Udo Kohlmeyer (JIRA)" <ji...@apache.org> on 2018/09/07 00:10:00 UTC

[jira] [Assigned] (GEODE-5705) InternalDataSerializer.basicReadObject convert IF to Switch

     [ https://issues.apache.org/jira/browse/GEODE-5705?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Udo Kohlmeyer reassigned GEODE-5705:
------------------------------------

    Assignee: Udo Kohlmeyer

> InternalDataSerializer.basicReadObject convert IF to Switch
> -----------------------------------------------------------
>
>                 Key: GEODE-5705
>                 URL: https://issues.apache.org/jira/browse/GEODE-5705
>             Project: Geode
>          Issue Type: Improvement
>          Components: serialization
>            Reporter: Udo Kohlmeyer
>            Assignee: Udo Kohlmeyer
>            Priority: Major
>             Fix For: 1.8.0
>
>
> In the InternalDataSerializer.basicReadObject, there are if statements for EVERY DSCODE defined. This has two major problems,
>  * The JIT compiler cannot easily improve this method, due to its size (1.3K)
>  * There is a worst case performance, that every condition is evaluated to read a serialized object.
> Converting this IF structure to switch the 2nd point is improved. The first point is reduced to 754bytes, which is still not optimal to inline but at least smaller and can still be improved.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)