[
https://issues.apache.org/jira/browse/FLINK-9503?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16504535#comment-16504535
]
ASF GitHub Bot commented on FLINK-9503:
---------------------------------------
Github user yanghua commented on a diff in the pull request:
https://github.com/apache/flink/pull/6129#discussion_r193707442
--- Diff:
flink-tests/src/test/java/org/apache/flink/test/iterative/aggregators/AggregatorsITCase.java
---
@@ -170,12 +162,12 @@ public void testAggregatorWithParameterForIterate()
throws Exception {
new NegativeElementsConvergenceCriterion());
DataSet<Integer> updatedDs = iteration.map(new
SubtractOneMapWithParam());
- iteration.closeWith(updatedDs).writeAsText(resultPath);
- env.execute();
+ List<Integer> result = iteration.closeWith(updatedDs).collect();
+ Collections.sort(result);
--- End diff --
@zentol the result need be sorted, otherwise the test would report error,
because the result is not ordered, print this :
```
java.lang.AssertionError:
Expected :[1, 2, 2, 3, 3, 3, 4, 4, 4, 4, 5, 5, 5, 5, 5]
Actual :[4, 2, 3, 5, 2, 5, 5, 5, 1, 3, 4, 4, 3, 5, 4]
```
> 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)
