-
Notifications
You must be signed in to change notification settings - Fork 31
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
Adding Builtin Functions #47
Labels
enhancement
New feature or request
Comments
grundprinzip
added a commit
that referenced
this issue
Aug 21, 2024
### What changes were proposed in this pull request? This patch adds basic support for generating the code of the basic builtin functions from PySpark in Go. ### Why are the changes needed? Compatibility ### Does this PR introduce _any_ user-facing change? Adds all PySpark builtin functions. ### How was this patch tested? Will add tests. Closes #44 from grundprinzip/adding_easy_builtin_functions. Authored-by: Martin Grund <martin.grund@databricks.com> Signed-off-by: Martin Grund <martin.grund@databricks.com>
haoxins
pushed a commit
to haoxins/spark-connect-go
that referenced
this issue
Aug 23, 2024
### What changes were proposed in this pull request? This patch adds basic support for generating the code of the basic builtin functions from PySpark in Go. ### Why are the changes needed? Compatibility ### Does this PR introduce _any_ user-facing change? Adds all PySpark builtin functions. ### How was this patch tested? Will add tests. Closes apache#44 from grundprinzip/adding_easy_builtin_functions. Authored-by: Martin Grund <martin.grund@databricks.com> Signed-off-by: Martin Grund <martin.grund@databricks.com>
Are there specific requirements for this enhancement ? Would you please enumerate which builtin functions you want to add ? thanks |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Today, the Go client does not have any builtin function support and we need to find a way to reliably keep up with Spark and Spark Connect.
The text was updated successfully, but these errors were encountered: