[THRIFT-4318] Delphi performance improvements

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.11.0
• Component/s: Delphi - Compiler
• Labels: perfomance
• Patch Info: Patch Available
• Assignee: Jens Geyer
• Reporter: Jens Geyer
• Created: 07/Sep/17 19:28
• Updated: 14/Dec/17 13:55
• Resolved: 07/Sep/17 20:15

Description

Misc. improvents for the Delphi implementation w/regard to overall performance. In the testcase used to develop this a performance boost of 20% or more has been observed esepcially with the binary protocol.

Note that my testcase is not part of the patch for several reasons and the total gains are also highly dependent on the data structures and the exact setup of the protocol/transport stack, so mileage may (and will) vary.

Important: The patch affects both compiler and library and is a breaking change vis-á-vis certain library structures (e.g. the interface IField is now a record called TThriftField). Therefore, all Thrift-generated Delphi code must be re-generated from the IDL to match the new library, otherwise compilation will not succeed.

Issue Links

• breaks: THRIFT-4323 range check errors or NPE in edge cases
• links to: GitHub Pull Request #1348