Github user fhueske commented on the pull request:
https://github.com/apache/flink/pull/1553#issuecomment-193272515
Hi Ram,
I just realized that the approach taken here might not work. We are
modifying the plan while it is enumerated. There might be cases, where this
leads to compiler errors or wrong plans. I have to check which side effects the
plan modification might have.
I would suggest we put this PR for a few days on ice and I check whether it
is possible to continue or if we have to find another approach.
---
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.
---
