[CRUNCH-674] CrunchRecordReader creates new TaskAttemptContextImpl with DummyReporter

• Type: Bug
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Affects Version/s: 0.12.0, 0.13.0, 0.14.0, 0.15.0
• Component/s: Core
• Assignee: Josh Wills
• Reporter: Stefan De Smit
• Created: 19/Oct/18 13:35
• Updated: 23/Oct/18 07:31

Description

CrunchRecordReader creates a new TaskAttemptContextImpl to pass to the actual RecordReader.

The constructor used, defaults to a DummyReporter.

The effect of this is that counters & progress messages from the RecordReader are lost.

This could easily be fixed by creating a WrappedStatusReporter that delegates counter & progress calls to the original context.