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

Updated DF (patched) to Dec-25, released ver44.0.0 #53

Open
wants to merge 8 commits into
base: base-df-dec25-ver44.0.0
Choose a base branch
from

Conversation

wiedld
Copy link
Collaborator

@wiedld wiedld commented Jan 9, 2025

Gets us up to commit apache@073a3b1, version 44.0.0 release.

Advances DF by these Dec-5 to Dec-25 commits: c510ab4...influxdata:arrow-datafusion:073a3b110852f97ccb7085ce4bfd19473b8a3f4f

Change in patches:

Removed 1 patch:

  • Screenshot 2025-01-08 at 9 32 27 AM

Added 3 patches:

  • Screenshot 2025-01-13 at 12 45 51 PM

Patches applied:

  • Patches already fixed in future upstream commits:

    • SanityCheckPlan patch
      • This is required until the CPU spinning bug is fixed. It was caused by the previous fix for the SanityCheckPlan, whcih updated the equivalence properties calculation for UNIONS.
    • Jan-10: pyo & ctor updates:
  • Patches still not fixed:

  • Patches specific to our DF branch (no need for upstream fix):

    • we are keeping in the warn logging for the schema mismatch. (Upstream only returns error, or not, based on schema misalignment): our patch 69d6118

@github-actions github-actions bot added documentation Improvements or additions to documentation sqllogictest core physical-expr common labels Jan 9, 2025
@alamb
Copy link
Collaborator

alamb commented Jan 10, 2025

Only clippy failures 😎 🚀

@alamb alamb marked this pull request as ready for review January 15, 2025 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
common core documentation Improvements or additions to documentation physical-expr sqllogictest
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants