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

Patched DataFusion, Oct 30, 2024 (almost) #44

Open
wants to merge 2 commits into
base: df-oct-30-base
Choose a base branch
from

Conversation

alamb
Copy link
Collaborator

@alamb alamb commented Nov 21, 2024

Not intended to merge, this PR documents the contents of the https://github.com/influxdata/arrow-datafusion/commits/df-upgrade-oct-30/ branch

Based on DataFusion at (almost) the end of day Oct 30 2024: 4975829

Note that I did not include this PR as I think it will need some special attention

Here are all the changes that went in upstream since #42

Patch list (note many fewer than on #44) - see commits

* Apply projection to `Statistics` in `FilterExec`

* Use Statistics::project in HashJoin
@alamb alamb changed the base branch from main to df-oct-30-base November 21, 2024 17:39
@alamb
Copy link
Collaborator Author

alamb commented Nov 21, 2024

Note that this removes the workaround for schema mismatches

So we should resolve / ensure we have a reproducer for the customer problem before we actually update to this version of DF

@github-actions github-actions bot added the core label Nov 21, 2024
wiedld pushed a commit that referenced this pull request Dec 6, 2024
…an (#44) (apache#12979)

* Don't preserve functional dependency when generating UNION logical plan

* Remove extra lines
alamb pushed a commit that referenced this pull request Dec 7, 2024
…an (#44) (apache#12979)

* Don't preserve functional dependency when generating UNION logical plan

* Remove extra lines
alamb pushed a commit that referenced this pull request Dec 7, 2024
…an (#44) (apache#12979)

* Don't preserve functional dependency when generating UNION logical plan

* Remove extra lines
wiedld pushed a commit that referenced this pull request Dec 12, 2024
…an (#44) (apache#12979)

* Don't preserve functional dependency when generating UNION logical plan

* Remove extra lines
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