Skip to content

Conversation

@andygrove
Copy link
Contributor

Creating this for a pairing session

@andygrove andygrove changed the title WIP: Datafusion rev 8ea59a [DF] WIP: Upgrade to DataFusion rev 8ea59a Sep 20, 2022
@ayushdg ayushdg marked this pull request as ready for review September 21, 2022 00:22
@codecov-commenter
Copy link

codecov-commenter commented Sep 21, 2022

Codecov Report

❗ No coverage uploaded for pull request base (datafusion-sql-planner@243b247). Click here to learn what that means.
The diff coverage is n/a.

@@                    Coverage Diff                    @@
##             datafusion-sql-planner     #779   +/-   ##
=========================================================
  Coverage                          ?   71.94%           
=========================================================
  Files                             ?       71           
  Lines                             ?     3572           
  Branches                          ?      743           
=========================================================
  Hits                              ?     2570           
  Misses                            ?      866           
  Partials                          ?      136           

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

Copy link
Collaborator

@charlesbluca charlesbluca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gpuCI seems to be slow right now; ran through the GPU tests locally and can verify that these changes resolved the remaining query failures

@charlesbluca charlesbluca merged commit 528108c into dask-contrib:datafusion-sql-planner Sep 21, 2022
@andygrove andygrove deleted the datafusion-rev-8ea59a branch September 21, 2022 15:14
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.

4 participants