-
Notifications
You must be signed in to change notification settings - Fork 13.6k
[FLINK-37797] Model DDL documentation #26694
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
base: master
Are you sure you want to change the base?
Conversation
**Input/Output Columns** | ||
|
||
The input columns define the features that will be used for model inference. The output columns define the predictions that the model will produce. Each column must have a name and data type. The input and output columns | ||
are optional because it can be inferred from training data later if Flink supports model training. For existing models, they are required to be defined. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When you say "later if Flink supports model training", do you mean that model training could be performed by the user at some later time, or that Flink might introduce this capability in some later offering. It is not clear.
without this context I do not understand what "For existing models, they are required to be defined." means.
DROP [TEMPORARY] MODEL [IF EXISTS] [catalog_name.][db_name.]model_name | ||
``` | ||
|
||
Drop a model with the given model name. If the model to drop does not exist, an exception is thrown. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this sentence is not true as it depends on [IF EXISTS] whether an Exception is thrown
|
||
The output includes the model name, model type, model options, and other configurations. This statement is useful when you need to understand the structure and configuration of an existing model or to recreate the model in another database. | ||
|
||
Assumes that the model `my_model` is created as follows: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I ams not sure what Assumes that the model
my_model is created as follows:
means . I assume my_model is just an example.
What is the purpose of the change
Documentation for model DDL
Verifying this change
Built doc locally
Does this pull request potentially affect one of the following parts:
@Public(Evolving)
: (yes / no)Documentation