[THRIFT-3006] Attach 'omitempty' json tag for optional fields in Go

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.9.3
• Component/s: Go - Compiler
• Patch Info: Patch Available
• Assignee: Jens Geyer
• Reporter: peter woodman
• Created: 21/Feb/15 02:11
• Updated: 03/Mar/15 20:51
• Resolved: 27/Feb/15 21:56

Description

This pull request improves on THRIFT-2549 in that it additionally tags optional fields with the omitempty tag so that dumping a struct to json doesn't result in a bunch of null fields. I think this is the right thing to do with optional fields, and additionally it matches the protoc behavior in go.