[GitHub] flink pull request: FLINK-2826 transformed is modified in Broadcas...

2015-11-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/1339 --- 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-2826 transformed is modified in Broadcas...

2015-11-11 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1339#issuecomment-155788263 Looks good. I hope we are not overlooking something (one never knows with these parallel synchronization cases), but it seems correct. All tests passed, so

[GitHub] flink pull request: FLINK-2826 transformed is modified in Broadcas...

2015-11-08 Thread tedyu
GitHub user tedyu opened a pull request: https://github.com/apache/flink/pull/1339 FLINK-2826 transformed is modified in BroadcastVariableMaterializatio… …n#decrementReferenceInternal without proper locking You can merge this pull request into a Git repository by running: