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

ASF GitHub Bot commented on FLINK-9503:
---------------------------------------

Github user zentol commented on a diff in the pull request:

    https://github.com/apache/flink/pull/6129#discussion_r193518671
  
    --- Diff: 
flink-tests/src/test/java/org/apache/flink/test/iterative/aggregators/AggregatorsITCase.java
 ---
    @@ -141,12 +133,17 @@ public void 
testAggregatorWithoutParameterForIterate() throws Exception {
                                new NegativeElementsConvergenceCriterion());
     
                DataSet<Integer> updatedDs = iteration.map(new 
SubtractOneMap());
    -           iteration.closeWith(updatedDs).writeAsText(resultPath);
    -           env.execute();
    +           List<Integer> result = iteration.closeWith(updatedDs).collect();
    +           Collections.sort(result);
    +
    +           List<Integer> expected = new ArrayList<Integer>() {{
    +                   add(-3); add(-2); add(-2); add(-1); add(-1);
    --- End diff --
    
    use `Arrays.asList` instead


> Migrate integration tests for iterative aggregators
> ---------------------------------------------------
>
>                 Key: FLINK-9503
>                 URL: https://issues.apache.org/jira/browse/FLINK-9503
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Tests
>            Reporter: Deepak Sharma
>            Assignee: vinoyang
>            Priority: Minor
>
> Migrate integration tests inĀ org.apache.flink.test.iterative.aggregators to 
> use collect() instead of temp files. Related to parent jira.



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

Reply via email to