[THRIFT-244] JSON Protocol doesn't extends TProtocolFactory and doesn't write messages begin properly

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Component/s: C++ - Library
• Patch Info: Patch Available
• Assignee: Jérémie BORDIER
• Reporter: Jérémie BORDIER
• Created: 29/Dec/08 21:10
• Updated: 01/Nov/11 02:51
• Resolved: 15/Jan/09 23:58

Description

The trunk cpp JSON protocol doesn't work because of two small issues:

• First part of the writeMessageBegin protocol is to write the Thrift Protocol version, which is stored in a uint8_t static variable. Unfortunately boost::lexical_cast handles uint8_t as a string type and doesn't write the given number. Fixed by moving the variable from uint8_t to uint32_t.

• The current JSONProtocolFactory implements TProtocolFactory but doesn't extend it, so it unuseable.