Skip to content
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

Support row-based Hive UDFs [databricks] #4224

Merged
merged 9 commits into from
Dec 2, 2021

Conversation

firestarman
Copy link
Collaborator

This fixes #3904

Signed-off-by: Firestarman firestarmanllc@gmail.com

Signed-off-by: Firestarman <firestarmanllc@gmail.com>
@firestarman
Copy link
Collaborator Author

Mark as draft, because needing to try an external app next.

@firestarman
Copy link
Collaborator Author

build

@firestarman firestarman changed the title Support row-based Hive UDFs Support row-based Hive UDFs [databricks] Nov 29, 2021
This UDF is an interface, so the only way is to suppress the warning.

Signed-off-by: Firestarman <firestarmanllc@gmail.com>
@firestarman
Copy link
Collaborator Author

build

Signed-off-by: Firestarman <firestarmanllc@gmail.com>
Signed-off-by: Firestarman <firestarmanllc@gmail.com>
Signed-off-by: Firestarman <firestarmanllc@gmail.com>
@firestarman
Copy link
Collaborator Author

build

1 similar comment
@firestarman
Copy link
Collaborator Author

build

@firestarman
Copy link
Collaborator Author

build

aggregator/pom.xml Outdated Show resolved Hide resolved
@firestarman firestarman marked this pull request as ready for review November 29, 2021 23:15
Signed-off-by: Firestarman <firestarmanllc@gmail.com>
@firestarman
Copy link
Collaborator Author

build

Copy link
Member

@jlowe jlowe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor suggestion on willNotWork wording but otherwise lgtm.

firestarman and others added 2 commits November 30, 2021 19:34
…HiveOverrides.scala


Update the willNotWork wording.

Co-authored-by: Jason Lowe <jlowe@nvidia.com>
…HiveOverrides.scala


Update the willNotWork wording for Hive GenericUDF.

Co-authored-by: Jason Lowe <jlowe@nvidia.com>
@firestarman
Copy link
Collaborator Author

build

@firestarman
Copy link
Collaborator Author

build

1 similar comment
@firestarman
Copy link
Collaborator Author

build

@firestarman firestarman merged commit 4bbdc28 into NVIDIA:branch-22.02 Dec 2, 2021
@firestarman firestarman deleted the hive-udf branch December 2, 2021 02:40
@sameerz sameerz added the feature request New feature or request label Dec 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update hive UDFs to support the feature of CPU based UDF
3 participants