MonetDB: Dec2023 - Check for nils.

[Sjoerd Mullender via checkin-list]

Changeset: 926f694c982c for MonetDB
URL: https://dev.monetdb.org/hg/MonetDB/rev/926f694c982c
Added Files:
        sql/test/BugTracker-2024/Tests/7528-jarowinlkler-null.test
Modified Files:
        monetdb5/modules/mal/txtsim.c
        sql/test/BugTracker-2024/Tests/All
Branch: Dec2023
Log Message:

Check for nils.

Fixes #7528.


diffs (34 lines):

diff --git a/monetdb5/modules/mal/txtsim.c b/monetdb5/modules/mal/txtsim.c
--- a/monetdb5/modules/mal/txtsim.c
+++ b/monetdb5/modules/mal/txtsim.c
@@ -569,6 +569,10 @@ TXTSIMjarowinkler(dbl *res, const char *
        str_item xi = { 0 }, yi = { 0 };
        str msg = MAL_SUCCEED;
 
+       if (strNil(*x) || strNil(*y)) {
+               *res = dbl_nil;
+               return MAL_SUCCEED;
+       }
        xi.val = *x;
        xi.len = UTF8_strlen(*x);
        if ((msg = str_2_codepointseq(&xi)) != MAL_SUCCEED)
diff --git a/sql/test/BugTracker-2024/Tests/7528-jarowinlkler-null.test 
b/sql/test/BugTracker-2024/Tests/7528-jarowinlkler-null.test
new file mode 100644
--- /dev/null
+++ b/sql/test/BugTracker-2024/Tests/7528-jarowinlkler-null.test
@@ -0,0 +1,7 @@
+statement ok
+CREATE TABLE t0(c1 VARCHAR)
+
+query T nosort
+SELECT * FROM t0 WHERE NOT NULL ORDER BY JAROWINKLER('a', NULL) DESC
+----
+
diff --git a/sql/test/BugTracker-2024/Tests/All 
b/sql/test/BugTracker-2024/Tests/All
--- a/sql/test/BugTracker-2024/Tests/All
+++ b/sql/test/BugTracker-2024/Tests/All
@@ -55,3 +55,4 @@ field-arg-error-Bug-7506
 7511-password-hash-missing-error
 7512-concurrent-globaltmp-instantiate-crash
 7513-uri-authority-parse-issue
+7528-jarowinkler-null
_______________________________________________
checkin-list mailing list -- checkin-list@monetdb.org
To unsubscribe send an email to checkin-list-le...@monetdb.org