[THRIFT-3306] Java: TBinaryProtocol: Use 1 temp buffer instead of allocating 8

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.10.0
• Component/s: Java - Library
• Patch Info: Patch Available
• Assignee: Roger Meier
• Reporter: Evan Jones
• Created: 25/Aug/15 15:50
• Updated: 04/Apr/16 18:52
• Resolved: 04/Apr/16 18:52

Description

TBinaryProtocol has a member buffer for reading and for serializing each of the integer types. It can get by with only a single one: 8 bytes long for the maximum length integer. This causes a significant reduction in allocations and GC in cases that create the TBinaryProtocol to serialize a single message, instead of reusing it.

This passes "ant test". I ran the existing SerializationBenchmark and there is no statistically significant difference (the average of 5 runs is smaller on my machine, but I don't think we should read much into that).

I have a JMH benchmark for this, and it shows that when the TBinaryProtocol is allocated each time, this change is better, but is more or less the same when you reuse the TBinaryProtocol in a loop.