Github user dawidwys commented on the issue:
https://github.com/apache/flink/pull/4121
Thanks a lot @dianfu for the work here. I had the last comment and will
merge it as soon as travis gives green.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user dianfu commented on the issue:
https://github.com/apache/flink/pull/4121
@dawidwys make sense, have updated the patch.
---
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 dawidwys commented on the issue:
https://github.com/apache/flink/pull/4121
Hi @dianfu, sorry for late reply. I was on short vacations. The updated PR
looks good! I agree though with @kl0u that changes from #4132 should be covered
by this PR.
---
If your project is set u
Github user dianfu commented on the issue:
https://github.com/apache/flink/pull/4121
Hi @dawidwys thanks a lot for the review, what's your thought for the
updated patch?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If