Github user mxm commented on the pull request:
https://github.com/apache/flink/pull/605#issuecomment-138863134
Hi @tammymendt. Thanks for the pull request! The accumulators work a little
bit different now because they are now accumulated on a per-task basis and
reported to the job manager in regular intervals.
The `clone()` method in `OperatorStatistics` doesn't create a deep copy of
the object, i.e. some references are reused. That causes problems when merging
the accumulators because runtime accumulators are modified while merging
accumulators for sending them to the job manager.
I could make the test pass by a nasty deep copy using Java serialization.
However, I didn't managed to make a proper copy using the provided interfaces.
I think you can probably do that faster because you know the code very well.
---
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.
---
