[HIVE-4877] In ExecReducer, remove tag from the row which will be passed to the first Operator at the Reduce-side

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.12.0
• Fix Version/s: 0.12.0
• Component/s: Query Processor
• Assignee: Yin Huai
• Reporter: Yin Huai
• Created: 17/Jul/13 16:13
• Updated: 15/Oct/13 23:29
• Resolved: 20/Jul/13 22:46

Description

In ExecReducer, we can see ...

row.clear();
row.add(keyObject);
row.add(valueObject[tag.get()]);
// The tag is not used any more, we should remove it.
row.add(tag);

I have seen this for a long time. Actually, we are using

reducer.process(row, tag.get());

to pass the tag.

If we do not need to add the tag to a row, we should remove it.

Issue Links

• links to: Review request