Github user StephanEwen commented on the issue:
https://github.com/apache/flink/pull/5892
Thanks, looks good, merging this...
---
Github user tedyu commented on the issue:
https://github.com/apache/flink/pull/5892
lgtm
---
Github user yanghua commented on the issue:
https://github.com/apache/flink/pull/5892
cc @zentol changed based on your suggestion
---
Github user yanghua commented on the issue:
https://github.com/apache/flink/pull/5892
cc @zentol @tzulitai
---
Github user yanghua commented on the issue:
https://github.com/apache/flink/pull/5892
cc @StephanEwen @tillrohrmann
---