[GEODE-6687] org.apache.geode.internal.ByteArrayDataInput can be used in place of ByteArrayInputStream+DataInputStream

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.10.0
• Component/s: serialization
• Labels: performance
• Assignee: Darrel Schneider
• Reporter: Darrel Schneider
• Created: 22/Apr/19 17:41
• Updated: 26/Sep/19 18:03
• Resolved: 03/May/19 22:02

Description

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.

 

Issue Links

• links to: GitHub Pull Request #3547