[ https://issues.apache.org/jira/browse/CRUNCH-674?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16659899#comment-16659899
]
Josh Wills commented on CRUNCH-674:
-----------------------------------
[~desmit] good point-- you're thinking something like the attached?
> CrunchRecordReader creates new TaskAttemptContextImpl with DummyReporter
> ------------------------------------------------------------------------
>
> Key: CRUNCH-674
> URL: https://issues.apache.org/jira/browse/CRUNCH-674
> Project: Crunch
> Issue Type: Bug
> Components: Core
> Affects Versions: 0.12.0, 0.13.0, 0.14.0, 0.15.0
> Reporter: Stefan De Smit
> Assignee: Josh Wills
> Priority: Minor
> Attachments: CRUNCH-674.patch
>
>
> 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.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
|