[THRIFT-894] Make default accessors for binary fields return byte[]; provide new accessors to get ByteBuffer version

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.5
• Assignee: Bryan Duxbury
• Reporter: Bryan Duxbury
• Created: 08/Sep/10 14:56
• Updated: 13/Sep/10 15:42
• Resolved: 13/Sep/10 15:42

Description

Folks have pointed out that it's not always best for users to interact with the ByteBuffers that are now backing binary fields in our structs. In truth, it seems like ByteBuffers are probably an expert means of access, with associated benefits and complexities.

I think we should generate two different sets of accessors for binary fields: a default set, named like usual, that return and take byte[], and a second set, named something like "get_buffer_for_<field>" that returns the underlying ByteBuffer. The byte[] accessors can wrap around the ByteBuffer ones as necessary.