[GitHub] flink issue #4784: FLINK-7736: fix some lgtm.com alerts

2017-12-17 Thread 1m2c3t4
Github user 1m2c3t4 commented on the issue: https://github.com/apache/flink/pull/4784 This has been reviewed and approved, but not committed. Can one of the committers take a look please ? ---

[GitHub] flink issue #4784: FLINK-7736: fix some lgtm.com alerts

2017-12-15 Thread 1m2c3t4
Github user 1m2c3t4 commented on the issue: https://github.com/apache/flink/pull/4784 Rebased again and fixed another obvious bug ---

[GitHub] flink issue #4784: FLINK-7736: fix some lgtm.com alerts

2017-10-12 Thread 1m2c3t4
Github user 1m2c3t4 commented on the issue: https://github.com/apache/flink/pull/4784 Rebased. @zentol @tillrohrmann thanks for reviewing this. Is it ready for merge now ? ---

[GitHub] flink issue #4784: FLINK-7736: fix some lgtm.com alerts

2017-10-09 Thread 1m2c3t4
Github user 1m2c3t4 commented on the issue: https://github.com/apache/flink/pull/4784 All review comments addressed ---