[GitHub] flink issue #3785: [FLINK-6337][network] Remove the buffer provider from Par...

2017-05-02 Thread uce
Github user uce commented on the issue: https://github.com/apache/flink/pull/3785 Thanks! Looks good to merge now 👍 I've rebased it and pushed it to Travis here: https://travis-ci.org/uce/flink/builds/227891373 As soon as it succeeds, I'm going to merge this PR. --- If

[GitHub] flink issue #3785: [FLINK-6337][network] Remove the buffer provider from Par...

2017-04-28 Thread zhijiangW
Github user zhijiangW commented on the issue: https://github.com/apache/flink/pull/3785 Hi @uce, I have submitted the modifications for the above two issues. Welcome any comments! :-) --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] flink issue #3785: [FLINK-6337][network] Remove the buffer provider from Par...

2017-04-27 Thread zhijiangW
Github user zhijiangW commented on the issue: https://github.com/apache/flink/pull/3785 Hi @uce , I have submitted the modifications of `createReadView`, and the tests and IT have passed in my private travis. I checked the failed test `HistoryServerTest.testFullArchiveLifecycle` on

[GitHub] flink issue #3785: [FLINK-6337][network] Remove the buffer provider from Par...

2017-04-27 Thread zhijiangW
Github user zhijiangW commented on the issue: https://github.com/apache/flink/pull/3785 Hey @uce , I have submitted the pull request for the issue we confirmed before. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If