[ 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