-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Description
Is your feature request related to a problem or challenge?
When we add a new function to datafusion's library we have to remember to document that function in the documentation, for example in https://datafusion.apache.org/user-guide/sql/scalar_functions.html
I observed this recently in #12429 (review). This likely means we have forgotten to document some functions or that the documentation has drifted over time
Also this means the help text for various functions can only be found on the DataFusion website, and not, for example within the function itself.
It would be awesome if you could do something like this from SQL:
> DESCRIBE sqrt;
Returns the square root of a number.
sqrt(numeric_expression)
Arguments
* numeric_expression: Numeric expression to operate on. Can be a constant, column, or function, and any combination of arithmetic operators.
Describe the solution you'd like
I would like:
- The help text / description of a ScalarUDFImpl (and AggregateUDFImpl) is available programatically (see add examples and description to scalar/aggregate functions? #8366)
- The contents of the SQL reference guide was automatically generated from the source code
DataFusion already does something like this for ConfigOptions
For example, the comments in https://docs.rs/datafusion/latest/datafusion/config/struct.SqlParserOptions.html are automatically added to the documentation programatically:
- Run this script: https://github.com/apache/datafusion/blob/main/dev/update_config_docs.sh
- Which runs this program:
// regarding copyright ownership. The ASF licenses this file - Which updates this file: https://github.com/apache/datafusion/blob/main/docs/source/user-guide/configs.md
Describe alternatives you've considered
I suggest this as a high level approach
- Add methods to the
ScalarUDFImpl
trait as proposed by @universalmind303 in add examples and description to scalar/aggregate functions? #8366ScalarUDFImpl::description
andScalarUDFImpl::sql_example
- Create a script and extraction program, similar to how it is does for
ConfigOptions
to generate the sql reference from those functions.
In terms of implementation order I would personally suggest breaking this project into smaller parts:
A first PR that does:
- Add the methods to the trait
- Move the documentation for one or two of the methods into the traits / code
- A script that creates some of the documentation (perhaps we could start by creating a new temporary page like https://datafusion.apache.org/user-guide/sql/scalar_functions_new.html that has only the auto generated documentation)
Then we can work in multiple PRs to port the remaining documentation over to the code (which will automatically result in the new page getting updated)
And then finally we can remove the old page when all functions are ported.
If we start working on this project, we (I) can file follow tickets to track porting the remaining functions / doing the same thing for aggregate functions, etc.
Additional context
Also, similarly, GlareDB has a way to automatically annotate functions with documentation, and @universalmind303 proposed something similar here #8366
Also, @findepi is considering implementing SHOW FUNCTIONS
as part of #12144 that could also likely take advantage of this documentation if it was present