feat: type resolution for lambdas using a new sql type #6967
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
feat: add functions as UDF input types fix immutable test proof of concept lambda follow up extend lambda functionality proof of concept lambda feat: add lambda syntax to grammar change g4 proof of concept lambda initial type checking bug fixes, clean up clean-up and adding lambda sql type stuff Type cleanup adding contexts cleanup updating qtt extending support for maps Updating for multiple input types Steven's ast fix + functional tests cleaning up
|
@confluentinc It looks like @lct45 just signed our Contributor License Agreement. 👍 Always at your service, clabot |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
resolves unknown types in lambdas through a new
SqlType.LAMBDAwhich allows us to store the input and return type of a lambda function and pass it to the UDF to properly identify the function. This PR would need to be updated to create a parent class for all regularSqlTypes andSqlLambda, so they become siblings of eachother, asSqlLambdadoes not need all the converters that a regularSqlTypewould.Testing done
Reviewer checklist