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
---