[GitHub] flink issue #5705: [FLINK-8699][FLINK-8968][state]Fix native resource leak c...

2018-04-05 Thread sihuazhou
Github user sihuazhou commented on the issue: https://github.com/apache/flink/pull/5705 Hi, @StefanRRichter thanks a lot. ---

[GitHub] flink issue #5705: [FLINK-8699][FLINK-8968][state]Fix native resource leak c...

2018-04-05 Thread StefanRRichter
Github user StefanRRichter commented on the issue: https://github.com/apache/flink/pull/5705 Thanks for the fixes. LGTM 👍 Will merge this. Sorry that this took so long. ---

[GitHub] flink issue #5705: [FLINK-8699][FLINK-8968][state]Fix native resource leak c...

2018-03-28 Thread sihuazhou
Github user sihuazhou commented on the issue: https://github.com/apache/flink/pull/5705 Hi @tillrohrmann thanks for your reply, and god bless Stefan, hope he will get well soon... ---

[GitHub] flink issue #5705: [FLINK-8699][FLINK-8968][state]Fix native resource leak c...

2018-03-28 Thread tillrohrmann
Github user tillrohrmann commented on the issue: https://github.com/apache/flink/pull/5705 Hi @sihuazhou, Stefan is currently sick but he should come back in a bit. Since he already reviewed parts of the PR, I would suggest to wait for his return. If he should not be back by next

[GitHub] flink issue #5705: [FLINK-8699][FLINK-8968][state]Fix native resource leak c...

2018-03-28 Thread sihuazhou
Github user sihuazhou commented on the issue: https://github.com/apache/flink/pull/5705 Could anyone have a look at this PR? ---