nseekhao opened a new pull request, #7612: URL: https://github.com/apache/arrow-datafusion/pull/7612
## Which issue does this PR close? <!-- We generally require a GitHub issue to be filed for all bug fixes and enhancements and this helps us generate change logs for our releases. You can link an issue to this PR using the GitHub syntax. For example `Closes #123` indicates that this PR will close issue #123. --> Closes #7611 . ## Rationale for this change For more efficient representation of join condition. Please refer to the related issue for more details. ## What changes are included in this PR? - Producer: instead of encoding the join `on` and `filter` separately in `expression` and `post_join_filter`, this PR encodes both in the `expression` field of the `JoinRel`. - Consumer: Adds a check for in each part of the conjunctive JoinRel expression. If the sub-expression is a `Column Eq Column` expression, then it gets added to the `on` field, otherwise, it gets AND-ed with the `filter` field. ## Are these changes tested? Yes. No new tests were added. Since the producer now hardcodes `post_join_filter` as `None`, if the changes are not working correctly, the existing would fail. ## Are there any user-facing changes? <!-- If there are user-facing changes then we may require documentation to be updated before approving the PR. --> No <!-- If there are any breaking changes to public APIs, please add the `api change` label. --> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
