Github user fhueske commented on the pull request:
https://github.com/apache/flink/pull/370#issuecomment-73913500
Hi @zentol,
I had a look at your PR and the failing test case. The test was in place to
ensure that objects are reused. For that the functions were implemented in a
way that they will produce wrong results if objects are reused (therefore the
unexpected result of 100 instead of 60). The problem was caused by the
CollectionInputFormat which sends out different objects (instead of reusing the
same object). When I changed that behavior and added some fixes to the
ChainedAllReduceDriver, all tests pass.
I will open a PR against your branch.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
