Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/2857
Thanks for the reviews and merging, @aljoscha @tillrohrmann . Closing this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/2857
Thanks for your work: ð
I merged it, could you please close this PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/2857
I introduced the planned change to the test. You could take another look if
you want :-)
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/2857
This was a preliminary PR, so that people can already take a look. However,
I think besides one sleep in a test, the added value is already enough to have
this in. Leftover todos can go to a
Github user aljoscha commented on the issue:
https://github.com/apache/flink/pull/2857
Overall, the changes and the tests look very good. There are some `TODO`s
left in the code, what about those?
---
If your project is set up for it, you can reply to this email and have your
reply