[GitHub] flink issue #3898: [FLINK-6492] Fix unclosed DataOutputViewStream usage

2017-05-23 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/3898 ah ... yes indeed, thanks for reminding me. I'll merge this to master and 1.3; it should be ok if it ends up missing in the 1.3.0 release. --- If your project is set up for it, you can reply to thi

[GitHub] flink issue #3898: [FLINK-6492] Fix unclosed DataOutputViewStream usage

2017-05-23 Thread zentol
Github user zentol commented on the issue: https://github.com/apache/flink/pull/3898 @tzulitai Did you forget about this PR? :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature en

[GitHub] flink issue #3898: [FLINK-6492] Fix unclosed DataOutputViewStream usage

2017-05-18 Thread zhangminglei
Github user zhangminglei commented on the issue: https://github.com/apache/flink/pull/3898 Hi, @huafengw Welcome also. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] flink issue #3898: [FLINK-6492] Fix unclosed DataOutputViewStream usage

2017-05-18 Thread huafengw
Github user huafengw commented on the issue: https://github.com/apache/flink/pull/3898 Thank you guys :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or

[GitHub] flink issue #3898: [FLINK-6492] Fix unclosed DataOutputViewStream usage

2017-05-18 Thread gallenvara
Github user gallenvara commented on the issue: https://github.com/apache/flink/pull/3898 Hi, @huafengw . Welcome to Flink family. Thanks for your PR and the changes look good to me. :) --- If your project is set up for it, you can reply to this email and have your reply appear on Git

[GitHub] flink issue #3898: [FLINK-6492] Fix unclosed DataOutputViewStream usage

2017-05-18 Thread tzulitai
Github user tzulitai commented on the issue: https://github.com/apache/flink/pull/3898 Hi @huafengw, thanks for the fixes! I'll probably merge this along with #3937 as there are some overlapping changes here and there. --- If your project is set up for it, you can reply to this email

[GitHub] flink issue #3898: [FLINK-6492] Fix unclosed DataOutputViewStream usage

2017-05-18 Thread huafengw
Github user huafengw commented on the issue: https://github.com/apache/flink/pull/3898 Hi @tzulitai, I'm new to Flink and would you mind taking a look at this one? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your p

[GitHub] flink issue #3898: [FLINK-6492] Fix unclosed DataOutputViewStream usage

2017-05-16 Thread huafengw
Github user huafengw commented on the issue: https://github.com/apache/flink/pull/3898 Can anyone take a look at this one? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled