You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@geode.apache.org by "xiaojian zhou (JIRA)" <ji...@apache.org> on 2016/12/02 00:15:58 UTC
[jira] [Resolved] (GEODE-2161) FilterRoutingInfo.FilterInfo.toData
does unneeded copying
[ https://issues.apache.org/jira/browse/GEODE-2161?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
xiaojian zhou resolved GEODE-2161.
----------------------------------
Resolution: Fixed
> FilterRoutingInfo.FilterInfo.toData does unneeded copying
> ---------------------------------------------------------
>
> Key: GEODE-2161
> URL: https://issues.apache.org/jira/browse/GEODE-2161
> Project: Geode
> Issue Type: Bug
> Reporter: xiaojian zhou
> Assignee: xiaojian zhou
>
> This is #52614
> FilterInfo.toData (that was being optimized for murex) does this:
> // create a hdos
> // write a bunch of stuff to it
> byte[] myData = hdos.toByteArray();
> DataSerializer.writeByteArray(myData, out);
> he toByteArray allocates a new byte array and copies all the data in the hdos to it.
> Then writeByteArray writes it all to "out".
> We can do this instead:
> if (out instanceof HeapDataOutputStream) {
> out.writeAsSerializedByteArray(hdos);
> } else {
> .. do it the old way
> }
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)