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

[REVIEW] Bump Arrow DataFusion Python dependency to 28.0.0 #1181

Merged
merged 104 commits into from
Aug 17, 2023

Conversation

jdye64
Copy link
Collaborator

@jdye64 jdye64 commented Jun 14, 2023

No description provided.

@jdye64
Copy link
Collaborator Author

jdye64 commented Jun 14, 2023

Seems like building and tests are failing due to some unrelated changes from this PR and rather some upstream pandas versioning from fugue. Will investigate and hopefully find a solution.

@jdye64 jdye64 marked this pull request as ready for review July 12, 2023 12:48
@jdye64 jdye64 changed the title Bump Arrow DataFusion Python dependency to 26.0.0 [REVIEW] Bump Arrow DataFusion Python dependency to 26.0.0 Jul 12, 2023
@jdye64 jdye64 changed the title [REVIEW] Bump Arrow DataFusion Python dependency to 26.0.0 [REVIEW] Bump Arrow DataFusion Python dependency to 27.0.0 Jul 12, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 13, 2023

Codecov Report

Merging #1181 (6e7c451) into main (3fad76b) will decrease coverage by 0.44%.
The diff coverage is 38.37%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main    #1181      +/-   ##
==========================================
- Coverage   82.13%   81.69%   -0.44%     
==========================================
  Files          78       78              
  Lines        4572     4610      +38     
  Branches      849      854       +5     
==========================================
+ Hits         3755     3766      +11     
- Misses        634      651      +17     
- Partials      183      193      +10     
Files Changed Coverage Δ
dask_sql/physical/rel/base.py 93.61% <0.00%> (ø)
dask_sql/physical/rel/convert.py 86.95% <0.00%> (ø)
dask_sql/physical/rel/custom/alter.py 90.00% <0.00%> (ø)
dask_sql/physical/rel/custom/analyze_table.py 88.00% <0.00%> (ø)
...k_sql/physical/rel/custom/create_catalog_schema.py 83.33% <0.00%> (ø)
...ask_sql/physical/rel/custom/create_memory_table.py 82.14% <0.00%> (ø)
dask_sql/physical/rel/custom/create_table.py 90.00% <0.00%> (ø)
dask_sql/physical/rel/custom/describe_model.py 86.95% <0.00%> (ø)
dask_sql/physical/rel/custom/distributeby.py 86.36% <0.00%> (ø)
dask_sql/physical/rel/custom/drop_schema.py 82.35% <0.00%> (ø)
... and 33 more

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@charlesbluca charlesbluca linked an issue Aug 17, 2023 that may be closed by this pull request
@charlesbluca charlesbluca merged commit 45e8bde into dask-contrib:main Aug 17, 2023
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

⚠️ Upstream CI Dask failed ⚠️
6 participants