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

ASF GitHub Bot commented on MAPREDUCE-6827:
-------------------------------------------

Github user javeme commented on the issue:

    https://github.com/apache/hadoop/pull/177
  
    NOTE: The following is a test about foreach with int[]/ArrayList, and the 
test results are expected(the second for-loop is also executed correctly):
    
        import java.util.ArrayList;
        
        public class TestForeach {
        
                public static void main(String[] args) {
                        
                        // test foreach twice with int[]
                        int list1[] = new int[]{1, 2};
                        
                        System.out.println("==== int[] 1");
                        for(int i : list1) {
                                System.out.println(i);
                        }
                        
                        System.out.println("===int[] 2");
                        for(int i : list1) {
                                System.out.println(i);
                        }
                        
                        // test foreach twice with ArrayList
                        ArrayList<String> list = new ArrayList<String>();
                        list.add("1");
                        list.add("2");
                        Iterable<String> list2 = list;
        
                        System.out.println();
                        System.out.println("===ArrayList 1");
                        for(String i : list2) {
                                System.out.println(i);
                        }
                        
                        System.out.println("===ArrayList 2");
                        for(String i : list2) {
                                System.out.println(i);
                        }
                }
        
        }



> Failed to traverse Iterable values the second time in reduce() method
> ---------------------------------------------------------------------
>
>                 Key: MAPREDUCE-6827
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-6827
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: task
>    Affects Versions: 3.0.0-alpha1
>         Environment: hadoop2.7.3
>            Reporter: javaloveme
>
> Failed to traverse Iterable values the second time in reduce() method
> The following code is a reduce() method (of WordCount):
> {code:title=WordCount.java|borderStyle=solid}
>       public static class WcReducer extends Reducer<Text, IntWritable, Text, 
> IntWritable> {
>               @Override
>               protected void reduce(Text key, Iterable<IntWritable> values, 
> Context context)
>                               throws IOException, InterruptedException {
>                       // print some logs
>                       List<String> vals = new LinkedList<>();
>                       for(IntWritable i : values) {
>                               vals.add(i.toString());
>                       }
>                       System.out.println(String.format(">>>> reduce(%s, 
> [%s])",
>                                       key, String.join(", ", vals)));
>                       // sum of values
>                       int sum = 0;
>                       for(IntWritable i : values) {
>                               sum += i.get();
>                       }
>                       System.out.println(String.format(">>>> reduced(%s, %s)",
>                                       key, sum));
>                       
>                       context.write(key, new IntWritable(sum));
>               }                       
>       }
> {code}
> After running it, we got the result that all sums were zero!
> After debugging, it was found that the second foreach-loop was not executed, 
> and the root cause was the returned value of Iterable.iterator(), it returned 
> the same instance in the two calls by foreach-loop. In general, 
> Iterable.iterator() should return a new instance in each call, such as 
> ArrayList.iterator().



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to