[HIVE-17576] Improve progress-reporting in TezProcessor

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.2.0, 2.4.0, 3.0.0
• Fix Version/s: 2.2.1, 2.4.0, 3.0.0
• Assignee: Mithun Radhakrishnan
• Reporter: Mithun Radhakrishnan
• Created: 22/Sep/17 00:07
• Updated: 22/May/18 23:15
• Resolved: 09/Oct/17 18:20

Description

Another one on behalf of selinazh and cdrome. Following the example in Apache Tez's MapProcessor, TezProcessor ought to use ProgressHelper to report progress for a Tez task. As per kshukla's advice,

Tez... provides getProgress() API for AbstractLogicalInput(s) which will give the correct progress value for a given Input. The TezProcessor(s) in Hive should use this to do something similar to what MapProcessor in Tez does today, which is use/override ProgressHelper to get the input progress and then set the progress on the processorContext.
...
The default behavior of the ProgressHelper class sets the processor progress to be the average of progress values from all inputs.

This code is whacked from inspired by MapProcessor's use of ProgressHelper.

(For my reference, YHIVE-978.)