[THRIFT-1768] unions can't have required fields (Compiler)

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.8, 0.9
• Fix Version/s: 0.9.1
• Component/s: Compiler (General)
• Patch Info: Patch Available
• Assignee: Jens Geyer
• Reporter: Jens Geyer
• Created: 29/Nov/12 21:52
• Updated: 08/Jun/13 03:15
• Resolved: 08/Jun/13 03:15

Description

Required fields within unions should issue a warning. First, required makes not much sense with unions. Next, most languages are already treating them as optional anyway, effectively ignoring the required attribute.

Issue Links

• relates to: THRIFT-1767 unions can't have required fields (Delphi)
• relates to: THRIFT-1769 unions can't have required fields (C++)