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

Feature/single source exec #56

Draft
wants to merge 13 commits into
base: apache_main
Choose a base branch
from

Conversation

mertak-synnada
Copy link

Which issue does this PR close?

Closes apache#13838.

Rationale for this change

This PR merges all Data sources into one Execution Plan, named DataSourceExec and a single trait named DataSource which is inspired by DataSink.

This version is not intended to be merged in Upstream since it removes all the ParquetExec, CsvExec, etc., and changes all the tests, but I'm sharing this as is so that we can see all possible changes. But our main intention is to re-adding old execution plans as deprecated ones and implement this part by part to keep API stability.

What changes are included in this PR?

Are these changes tested?

Are there any user-facing changes?

@mertak-synnada mertak-synnada changed the title Chore/single source exec Feature/single source exec Jan 7, 2025
@mertak-synnada mertak-synnada marked this pull request as draft January 7, 2025 14:27
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.

1 participant