Github user pnowojski commented on the issue:
https://github.com/apache/flink/pull/4915
Fine for me, thanks!
---
Github user tzulitai commented on the issue:
https://github.com/apache/flink/pull/4915
I've added a commit to rename the 011 producer integration test to
`ITCase`. Please let me know if you disagree with that @pnowojski @GJL.
---
Github user tzulitai commented on the issue:
https://github.com/apache/flink/pull/4915
Thanks for the work and reviews @GJL @pnowojski.
I did a pass also, changes LGTM.
Left one last comment regarding the integration test class name.
Other than that, would like to merge
Github user GJL commented on the issue:
https://github.com/apache/flink/pull/4915
ð
---
Github user pnowojski commented on the issue:
https://github.com/apache/flink/pull/4915
@GJL @aljoscha could you take a look?
---
