[GitHub] flink pull request: [FLINK-2540] [optimizer] [runtime] Propagate u...

2015-08-21 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/1036 --- 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 enab

[GitHub] flink pull request: [FLINK-2540] [optimizer] [runtime] Propagate u...

2015-08-21 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1036#issuecomment-133363161 Merging this, looks good, test works and catches the problematic condition! --- If your project is set up for it, you can reply to this email and have your reply app

[GitHub] flink pull request: [FLINK-2540] [optimizer] [runtime] Propagate u...

2015-08-21 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1036#issuecomment-133358126 Ah, the github mirror is out of sync again - that's why this pull request includes bac21bf5d77c8e15c608ecbf006d29e7af1dd68a again (which is in the apache master, but

[GitHub] flink pull request: [FLINK-2540] [optimizer] [runtime] Propagate u...

2015-08-21 Thread StephanEwen
Github user StephanEwen commented on the pull request: https://github.com/apache/flink/pull/1036#issuecomment-11346 Other then the branch mixup, this looks good. Will test it and merge it, if it works... --- If your project is set up for it, you can reply to this email an

[GitHub] flink pull request: [FLINK-2540] [optimizer] [runtime] Propagate u...

2015-08-20 Thread tillrohrmann
Github user tillrohrmann commented on the pull request: https://github.com/apache/flink/pull/1036#issuecomment-132948630 Is it intended that this PR is based on FLINK-2398? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] flink pull request: [FLINK-2540] [optimizer] [runtime] Propagate u...

2015-08-19 Thread uce
GitHub user uce opened a pull request: https://github.com/apache/flink/pull/1036 [FLINK-2540] [optimizer] [runtime] Propagate union batch exchanges to union inputs The DataExchangeMode of union nodes was not respected when translating an OptimizedPlan to a JobGraph. This could resu