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 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 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 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 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 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