[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 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 if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 this travis and it seems no 
related to my pull request. I run it several times separately again and all are 
passed.


---
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 if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[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 your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---