Skip to content

Conversation

@charlesbluca
Copy link
Collaborator

Generalizes CREATE | PREDICT MODEL to accept DaskStatements instead of SQLStatements, so that we can do things like:

SELECT * FROM PREDICT (
    MODEL my_model,
    SELECT x, y FROM PREDICT (
        ...
    )
)

This is something that would've happened in working on FILTER WHERE support, but figured it might make sense to break this out into its own PR.

@codecov-commenter
Copy link

codecov-commenter commented Sep 8, 2022

Codecov Report

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

@@                    Coverage Diff                    @@
##             datafusion-sql-planner     #747   +/-   ##
=========================================================
  Coverage                          ?   74.89%           
=========================================================
  Files                             ?       73           
  Lines                             ?     3657           
  Branches                          ?      760           
=========================================================
  Hits                              ?     2739           
  Misses                            ?      776           
  Partials                          ?      142           

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

@charlesbluca charlesbluca marked this pull request as ready for review September 13, 2022 15:43
@galipremsagar galipremsagar merged commit da1b31f into dask-contrib:datafusion-sql-planner Sep 19, 2022
@charlesbluca charlesbluca deleted the df-nested-model-parsing branch February 5, 2024 19:43
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