Closed xzhseh closed 6 months ago
LGTM. But I prefer to complete a minimal workable SQL UDF in a single PR. This way we can revisit the whole design and see where can be optimized or simplified. 🤔
For sure, I'll add named sql udf support in this PR later. (anonymous sql udf support in the future)
We now support named sql udf in risinglight!
e.g., the following definitions are totally valid and could be used as normal built-in functions now.
See the test cases for detailed use cases.
P.S. anonymous sql udf may be supported in subsequent PRs, with other necessary preparations.
cc @wangrunji0408.