Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ARROW-14350: [IR] Add filter expression to Source node #11438

Closed
wants to merge 2 commits into from

Conversation

cpcloud
Copy link
Contributor

@cpcloud cpcloud commented Oct 15, 2021

This PR adds a filter expression to Source nodes to support
consumers that implement predicate pushdown.

@cpcloud cpcloud requested a review from bkietz October 15, 2021 17:53
@github-actions
Copy link

Copy link
Member

@bkietz bkietz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@bkietz bkietz closed this in 176c113 Oct 22, 2021
@ElenaHenderson
Copy link
Contributor

ElenaHenderson commented Oct 22, 2021

Benchmark runs are scheduled for baseline = 9ce3440 and contender = 176c113. 176c113 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Finished ⬇️0.0% ⬆️0.0%] ec2-t3-xlarge-us-east-2
[Scheduled] ursa-i9-9960x
[Finished ⬇️0.09% ⬆️0.04%] ursa-thinkcentre-m75q
Supported benchmarks:
ursa-i9-9960x: langs = Python, R, JavaScript
ursa-thinkcentre-m75q: langs = C++, Java
ec2-t3-xlarge-us-east-2: cloud = True

ViniciusSouzaRoque pushed a commit to s1mbi0se/arrow that referenced this pull request Nov 3, 2021
This PR adds a `filter` expression to `Source` nodes to support
consumers that implement predicate pushdown.

Closes apache#11438 from cpcloud/ARROW-14350

Authored-by: Phillip Cloud <417981+cpcloud@users.noreply.github.com>
Signed-off-by: Benjamin Kietzman <bengilgit@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants