Github user fhueske commented on the pull request:
https://github.com/apache/flink/pull/1981#issuecomment-221926857
Thanks for looking into how Calcite splits the join conditions. You are
right, we do not need to be "smarter" than Calcite and approve predicates that
will be later rejected. So +1 for your suggestion.
Apart from the documentation update I would only extend the error messages
with the conditions that cause the failure. After that the PR should be good to
merge.
---
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.
---
