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

Closed
wants to merge 2 commits into 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
@wiedld
Copy link
Collaborator

wiedld commented Dec 12, 2024

Closing this in favor of updated:
#48

@wiedld wiedld closed this Dec 12, 2024
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.

2 participants