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

[C++][Compute] Promote Expression to the compute namespace #27766

Closed
asfimport opened this issue Mar 10, 2021 · 1 comment
Closed

[C++][Compute] Promote Expression to the compute namespace #27766

asfimport opened this issue Mar 10, 2021 · 1 comment

Comments

@asfimport
Copy link
Collaborator

asfimport commented Mar 10, 2021

See discussion in https://docs.google.com/document/d/1AyTdLU-RxA-Gsb9EsYnrQrmqPMOYMfPlWwxRi1Is1tQ

This will be necessary for ARROW-11928 since some ExecNodes will be constructed with one or more Expressions. For example FilterNodes will evaluate an Expression against their input batches to produce selection vectors.

This should probably include factoring out dataset-specific considerations such as allowing input batches diverge slightly in schema. Bound Expressions should refer to their parameters by index so that by-name lookup can be avoided.

Reporter: Ben Kietzman / @bkietz
Assignee: Ben Kietzman / @bkietz

Related issues:

PRs and other links:

Note: This issue was originally created as ARROW-11929. Please see the migration documentation for further details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants