-
Notifications
You must be signed in to change notification settings - Fork 232
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
Arbitrary function call in UDF #854
Conversation
@seanprime7 Added initial test case for this feature. Current exception: Issue created: #855 |
It would be better to create an issue for this too. @wjxiz1992 |
691e5b4
to
ca62ffe
Compare
@wjxiz1992 @nvnedelman do you guys see if there is any other method call pattern that needs to be added to the test suite? |
Signed-off-by: Allen Xu <allxu@nvidia.com>
ca62ffe
to
0bcfc17
Compare
Resolved merge conflicts. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just have a couple of nits @seanprime7. Looks good to me though.
With this change, method calls are inlined only if the method being called 1. consists of operations supported by the UDF compiler, and 2. is one of the folllowing: * a final method, or * a method in a final class, or * a method in a final class Signed-off-by: Sean Lee <selee@nvidia.com>
0bcfc17
to
b37dba8
Compare
build |
* Add test case for arbitrary function call in UDF Signed-off-by: Allen Xu <allxu@nvidia.com> * Inline method calls when possible. With this change, method calls are inlined only if the method being called 1. consists of operations supported by the UDF compiler, and 2. is one of the folllowing: * a final method, or * a method in a final class, or * a method in a final class Signed-off-by: Sean Lee <selee@nvidia.com> Co-authored-by: Allen Xu <allxu@nvidia.com> Co-authored-by: Sean Lee <selee@nvidia.com>
* Add test case for arbitrary function call in UDF Signed-off-by: Allen Xu <allxu@nvidia.com> * Inline method calls when possible. With this change, method calls are inlined only if the method being called 1. consists of operations supported by the UDF compiler, and 2. is one of the folllowing: * a final method, or * a method in a final class, or * a method in a final class Signed-off-by: Sean Lee <selee@nvidia.com> Co-authored-by: Allen Xu <allxu@nvidia.com> Co-authored-by: Sean Lee <selee@nvidia.com>
* Add test case for arbitrary function call in UDF Signed-off-by: Allen Xu <allxu@nvidia.com> * Inline method calls when possible. With this change, method calls are inlined only if the method being called 1. consists of operations supported by the UDF compiler, and 2. is one of the folllowing: * a final method, or * a method in a final class, or * a method in a final class Signed-off-by: Sean Lee <selee@nvidia.com> Co-authored-by: Allen Xu <allxu@nvidia.com> Co-authored-by: Sean Lee <selee@nvidia.com>
…IDIA#854) Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com> Signed-off-by: spark-rapids automation <70000568+nvauto@users.noreply.github.com>
Enable arbitrary function call in UDF
Signed-off-by: Allen Xu allxu@nvidia.com