[HIVE-16184] Remove use of optionals in union; declare TProgressUpdateResp before its usage

• Type: Bug
• Status: Patch Available
• Priority: Minor
• Resolution: Unresolved
• Component/s: Thrift API
• Labels: pull-request-available
• Assignee: Ian Macalinao
• Reporter: Ian Macalinao
• Created: 12/Mar/17 23:07
• Updated: 13/May/19 07:34

Description

A Scala Thrift generator (https://github.com/twitter/scrooge) does not like the Thrift file. I have removed some bad Thrift practices (i.e. optionals in union, dependent types declared after their usage) and replaced them with things that work in Scrooge.

See https://github.com/apache/hive/pull/157

Issue Links

• links to: GitHub Pull Request #157