[GitHub] flink pull request: [FLINK-1775] BarrierBuffer fix to avoid end of...

2015-03-26 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/534 --- 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

[GitHub] flink pull request: [FLINK-1775] BarrierBuffer fix to avoid end of...

2015-03-26 Thread gyfora
Github user gyfora commented on the pull request: https://github.com/apache/flink/pull/534#issuecomment-86554780 Now it seems that all these issues are fixed. --- 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

[GitHub] flink pull request: [FLINK-1775] BarrierBuffer fix to avoid end of...

2015-03-25 Thread uce
Github user uce commented on the pull request: https://github.com/apache/flink/pull/534#issuecomment-85909585 Hey gyula, What is the problem with the stream writer? And how can I reproduce the record writer broadcast problem? If you give me some instructions to reproduce the

[GitHub] flink pull request: [FLINK-1775] BarrierBuffer fix to avoid end of...

2015-03-25 Thread rmetzger
Github user rmetzger commented on a diff in the pull request: https://github.com/apache/flink/pull/534#discussion_r27100141 --- Diff: flink-staging/flink-streaming/flink-streaming-core/src/main/java/org/apache/flink/streaming/api/streamvertex/StreamVertex.java --- @@ -283,16

[GitHub] flink pull request: [FLINK-1775] BarrierBuffer fix to avoid end of...

2015-03-25 Thread gyfora
Github user gyfora commented on a diff in the pull request: https://github.com/apache/flink/pull/534#discussion_r27115550 --- Diff: flink-staging/flink-streaming/flink-streaming-core/src/main/java/org/apache/flink/streaming/api/streamvertex/StreamVertex.java --- @@ -283,16

[GitHub] flink pull request: [FLINK-1775] BarrierBuffer fix to avoid end of...

2015-03-25 Thread gyfora
Github user gyfora commented on the pull request: https://github.com/apache/flink/pull/534#issuecomment-86140841 Should we merge this anyway? The recordwriter issue seems to be independent from this, and this commit solves some other bugs. --- If your project is set up for it, you

[GitHub] flink pull request: [FLINK-1775] BarrierBuffer fix to avoid end of...

2015-03-24 Thread gyfora
GitHub user gyfora opened a pull request: https://github.com/apache/flink/pull/534 [FLINK-1775] BarrierBuffer fix to avoid end of stream deadlock This fixes several corner-cases in the barrier buffer logic. Still two problems remain: - If the StreamRecordWriter is