Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/3081
Changes look good to me. Thanks a lot for your contributions @Fokko.
Merging this PR.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user Fokko commented on the issue:
https://github.com/apache/flink/pull/3081
I've rebased both branches with master.
---
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
Github user Fokko commented on the issue:
https://github.com/apache/flink/pull/3081
Hi @tillrohrmann, did you find any time to check #3077 and 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 your project does
Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/3081
Thanks for your contribution @Fokko. I'll take a look at your PR in the
next days.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user Fokko commented on the issue:
https://github.com/apache/flink/pull/3081
I think we have some flakey tests, since it passes on my own travis:
https://travis-ci.org/Fokko/flink/builds/189854940
---
If your project is set up for it, you can reply to this email and have
Github user Fokko commented on the issue:
https://github.com/apache/flink/pull/3081
I think we have some flakey tests, since it passes on my own travis:
https://travis-ci.org/Fokko/flink/builds/189855914
---
If your project is set up for it, you can reply to this email and have
