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

R UDFs with empty run function hangs #754

Closed
stmandl opened this issue Feb 3, 2023 · 1 comment · Fixed by #779
Closed

R UDFs with empty run function hangs #754

stmandl opened this issue Feb 3, 2023 · 1 comment · Fixed by #779
Assignees
Labels
bug Unwanted / harmful behavior

Comments

@stmandl
Copy link
Contributor

stmandl commented Feb 3, 2023

This UDF does not terminate and eventually runs out of memory:

create schema test;

create or replace R scalar script test.dummy()
returns int AS 
run <- function(ctx) {
}
/


select test.dummy();
@redcatbear redcatbear added the bug Unwanted / harmful behavior label Feb 13, 2023
@tkilias tkilias self-assigned this Mar 21, 2023
@tkilias
Copy link
Collaborator

tkilias commented Mar 21, 2023

The problem occurs, because an empty function returns NULL. I tried to return NULL directly and got the same error. We probably expect a vector as return value, and everything in R is a vector except NULL. Also, a vector of NULL is NULL

@MarleneKress79789 MarleneKress79789 added the blocked:yes Currently blocked by another ticket label Mar 23, 2023
@tkilias tkilias removed the blocked:yes Currently blocked by another ticket label Mar 30, 2023
tkilias added a commit that referenced this issue Mar 31, 2023
* Fixed return behavior of SCALAR RETURNS UDFs for null.
* Added tests for various edge cases of return values
@tkilias tkilias closed this as completed Mar 31, 2023
tkilias added a commit that referenced this issue Mar 31, 2023
Changelog:

* #762: Updated ubuntu/pip packages (#763)

* Reformat scripts/update_apt_package_in_package_list.sh
* Make cryptography 39.0.2 and openssl 3.0.8 working with conda by updating mamba
* Update ubuntu packages
* Fix package versions in install script tests
* Update pyOpenSSL package to be compatible with new cryptography package for pip packages

Co-authored-by: Torsten Kilias <torsten.kilias@exasol.com>

* Remove python2 from the flavors (#766)

* Remove python2 from the flavors
* Remove python2 from udfclient
* Remove smb from standard 7.* flavors to remove python2 completely
* Update smb for standard 8.0.0

---------

Co-authored-by: Thomas Ubensee <34603111+tomuben@users.noreply.github.com>

* Update libprotobuf-dev (#769)

* Fix language definiton for standard 7.0 flavor (#771)

* #775: Updated dependencies (#776)

* Updated ubuntu packages 
* Pinned icu and libxml2 conda packages to be able to install openjdk11 with conda
* Update redis-py to 4.5.3 to fix CVE-2023-28858
* Ignore CVE-2023-28859, because there is no fix yet

* #774: Updated CI to the version released on PyPi (#778)

* Update CI and reorganize tests directory to fit the new layout of the test_container directory

* #768: Removed or replaced Python 2 tests (#773)

* Fix testconfig for standard-EXASOL-7.* flavor and updated submodule

* #754: Fixed R SCALAR RETURNS UDFs with empty run function hangs (#779)

* Fixed return behavior of SCALAR RETURNS UDFs for null.
* Added tests for various edge cases of return values

* #780: Prepared release (#781)

---------

Co-authored-by: Thomas Ubensee <34603111+tomuben@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unwanted / harmful behavior
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants