-
Notifications
You must be signed in to change notification settings - Fork 1
initial update of UDF sections #12
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: main
Are you sure you want to change the base?
Conversation
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.
Minor language suggestions. Overall, the content looks good to me. If you're looking for more advanced feedback, I'd recommend waiting for a more experienced team member ;)
Co-authored-by: Ariel Schulz <[email protected]>
Co-authored-by: Ariel Schulz <[email protected]>
Co-authored-by: Ariel Schulz <[email protected]>
Co-authored-by: Ariel Schulz <[email protected]>
Co-authored-by: Ariel Schulz <[email protected]>
Co-authored-by: Ariel Schulz <[email protected]>
Co-authored-by: Ariel Schulz <[email protected]>
Co-authored-by: Mikhail Beck <[email protected]>
update as per syntax feedback first round |
------------------------- | ||
|
||
For best performance, we recommend that you create a script using CREATE SCRIPT and then use this script within a SELECT statement. Embedding the script within SQL in this way provides the best performance and scalability. |
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.
Sorry, I should have picked it in the first pass. I don't know any other way of creating a UDF other than executing the CREATE SCRIPT
command. These two sentences do not make a lot of sense to me.
Added initial documentation for UDFs - will need further work over time including examples