[THRIFT-3363] Reporting a missing required field (TProtocolException::INVALID_DATA) to the client is not informative enough

• Type: Improvement
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 0.8, 0.9, 0.9.1, 0.9.2, 0.9.3
• Component/s: C++ - Compiler, C++ - Library
• Assignee: Unassigned
• Reporter: James E. King III
• Created: 30/Sep/15 13:36
• Updated: 22/Jun/16 02:49

Description

This may affect all languages, however it was initially reported against C++ by ggodik in THRIFT-2342:

Every time a required field is not set, the exception that gets generated is a very generic:
throw TProtocolException(TProtocolException::INVALID_DATA);

(see - https://git-wip-us.apache.org/repos/asf?p=thrift.git;a=blob;f=compiler/cpp/src/generate/t_cpp_generator.cc;h=298096d3265baa1f008501eb7c26bb4ae96ffa4b;hb=HEAD#l1372)

It would be useful to indicate the invalid data field in this exception case.

Issue Links

• is a clone of: THRIFT-2342 Add __FILE__ and __LINE__ to Thrift C++ excpetions