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 7, 2024 #45

Closed
wants to merge 10 commits into from
Closed

Conversation

alamb
Copy link
Collaborator

@alamb alamb commented Dec 6, 2024

This PR is not intended to merge.

This PR documents the contents of the https://github.com/influxdata/arrow-datafusion/commits/df-upgrade-oct-7/ branch

Based on DataFusion at the end of day Oct 7 2024: 577e4bb

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

Patch list:

  • You can see the changes by looking at the commits on this PR

@wiedld
Copy link
Collaborator

wiedld commented Dec 6, 2024

@alamb -- let me know if I should review this one.

Note that we'll also need to add in this upstream DF SHA/patch to any future DF upgrade.

@alamb
Copy link
Collaborator Author

alamb commented Dec 6, 2024

@alamb -- let me know if I should review this one.

Thanks @wiedld -- I just want to check a few things first

Note that we'll also need to add in this upstream DF SHA/patch to any future DF upgrade.

Got it -- can we add a reproducer into iox so we don't forget?

@alamb alamb marked this pull request as draft December 6, 2024 19:04
@alamb
Copy link
Collaborator Author

alamb commented Dec 6, 2024

Will mark read to review when ready for a look

@alamb
Copy link
Collaborator Author

alamb commented Dec 6, 2024

I want to ensure all CI tests pass specifically

@alamb alamb marked this pull request as ready for review December 6, 2024 22:23
…an (#44) (apache#12979)

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

* Remove extra lines
@alamb
Copy link
Collaborator Author

alamb commented Dec 7, 2024

@alamb -- let me know if I should review this one.

Note that we'll also need to add in this upstream DF SHA/patch to any future DF upgrade.

I have added this to this branch

@wiedld
Copy link
Collaborator

wiedld commented Dec 12, 2024

No longer used in iox. Closing.

@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.

3 participants