[
https://issues.apache.org/jira/browse/GEODE-6687?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16832666#comment-16832666
]
ASF subversion and git services commented on GEODE-6687:
--------------------------------------------------------
Commit 4890493feacac15467be6658614e28ba240d3557 in geode's branch
refs/heads/develop from Darrel Schneider
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=4890493 ]
Revert "GEODE-6687: replace ByteArrayInputStream+DataInputStream with
ByteArrayDataInput"
This reverts commit 8c2c9c82fc5d09e643e245676e77b0a64c6c158a.
> org.apache.geode.internal.ByteArrayDataInput can be used in place of
> ByteArrayInputStream+DataInputStream
> ---------------------------------------------------------------------------------------------------------
>
> Key: GEODE-6687
> URL: https://issues.apache.org/jira/browse/GEODE-6687
> Project: Geode
> Issue Type: Improvement
> Components: serialization
> Reporter: Darrel Schneider
> Assignee: Darrel Schneider
> Priority: Major
> Labels: performance
> Fix For: 1.10.0
>
>
> A number of places in geode wrap a byte array in a ByteArrayInputStream which
> then is given to a DataInputStream. Because the DataInputStream always
> allocates some byte arrays and char arrays (even if they will not be used)
> this can generate a good deal of extra garbage.
> Geode has an existing class, org.apache.geode.internal.ByteArrayDataInput,
> that can be used to replace a ByteArrayInputStream+DataInputStream and will
> generate less garbage.
>
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
