[THRIFT-2812] Go server adding redundant buffering layer

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.9.2
• Fix Version/s: 0.9.3
• Component/s: Go - Library
• Patch Info: Patch Available
• Assignee: Jens Geyer
• Reporter: Stephen Marbowitz
• Created: 10/Nov/14 22:15
• Updated: 22/Nov/14 21:16
• Resolved: 10/Nov/14 23:15

Description

THRIFT-2791 incorrectly assumed there was no way to do buffered sockets on the server. The solution is to simply use a TBufferedTransportFActory and give it to the server at creation time. Can we please revert that patch? Any code that was previously using buffered server sockets now has an extra layer of buffering. I consider this a breaking change.

Issue Links

• depends upon: THRIFT-2791 Allowing use of buffered sockets in go server