[THRIFT-5279] Cleanups/small optimizations for go's serializer/deserializer code

• Type: Task
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 0.13.0
• Fix Version/s: 0.14.0
• Component/s: Go - Library
• Assignee: Yuxuan Wang
• Reporter: Yuxuan Wang
• Created: 22/Sep/20 21:44
• Updated: 23/Sep/20 16:17
• Resolved: 23/Sep/20 16:17

Description

Currently go's default NewTSerializer and NewTDeserializer implementations use NewTBinaryProtocolFactoryDefault().GetProtocol(transport) to get the protocol. This is essentially the same as NewTBinaryProtocolTransport(transport), but with an extra allocation of the ProtocolFactory that is only used once then discarded. Streamline it will give as a minor performance gain.

Also I plan to add new constructors for TSerializerPool and TDeserializerPool to make it easier to use with non-default protocols.

Issue Links

• links to: GitHub Pull Request #2242