Github user zentol commented on the issue:
https://github.com/apache/flink/pull/4698
merging.
---
Github user zhijiangW commented on the issue:
https://github.com/apache/flink/pull/4698
@zentol , I have rebased the latest master codes and solved the conflicts.
---
Github user NicoK commented on the issue:
https://github.com/apache/flink/pull/4698
ok, cool, then we can merge this PR as anticipated
---
Github user zhijiangW commented on the issue:
https://github.com/apache/flink/pull/4698
@NicoK , it already passed travis tests this time.
---
Github user zhijiangW commented on the issue:
https://github.com/apache/flink/pull/4698
@NicoK , thanks for your reviews!
I checked the travis failure before and thought it should be existing flaky
ones. I will re-trigger the tests later and see the results.
---
Github user NicoK commented on the issue:
https://github.com/apache/flink/pull/4698
Actually, what worries me a bit are the test failures of the 3rd and 4th
test profile. I think, they're unrelated (especially the Kafka ones which are
kind of instable at the moment in general), but th