[ 
https://issues.apache.org/jira/browse/MAPREDUCE-7072?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16448958#comment-16448958
 ] 

Robert Kanter commented on MAPREDUCE-7072:
------------------------------------------

Great investigation [~wilfreds]!  The patch looks good overall, just one thing:
# Instead of using the iterator:
{code:java}
Iterator<CounterGroup> groupNames = totalCounters.iterator();
while (groupNames.hasNext()) {
     String groupName = groupNames.next().getName();
{code}
I think we can still do a for-each loop like this:
{code:java}
for (CounterGroup counterGroup : totalCounters) {
     String groupName = counterGroup.getName();
{code}

> mapred job -history prints duplicate counter in human output
> ------------------------------------------------------------
>
>                 Key: MAPREDUCE-7072
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-7072
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: client
>    Affects Versions: 3.0.0
>            Reporter: Wilfred Spiegelenburg
>            Assignee: Wilfred Spiegelenburg
>            Priority: Major
>         Attachments: MAPREDUCE-7072.01.patch
>
>
>  'mapred job -history' command prints duplicate entries for counters only for 
> the human output format. It does not do this for the JSON format.
> mapred job -history /user/history/somefile.jhist -format human
> {code}
> ....
> |Job Counters |Total megabyte-seconds taken by all map tasks|0 |0 |268,288,000
> ...
> |Job Counters |Total megabyte-seconds taken by all map tasks|0 |0 |268,288,000
> ....
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: mapreduce-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: mapreduce-issues-h...@hadoop.apache.org

Reply via email to