Fix the result mismatch for right expression (#107)

oap-project · Feb 22, 2023 · 7c73eb5 · 7c73eb5
1 parent 91ddb55
commit 7c73eb5
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/velox/functions/prestosql/StringFunctions.h b/velox/functions/prestosql/StringFunctions.h
@@ -325,6 +325,13 @@ struct SubstrFunction {
     if (start < 0) {
       start = numCharacters + start + 1;
     }
+
+    // Following vanilla spark semantics. When the start is 0 after adjusing start,
+    // Keep the origin input. 
+    if (start <= 0) {
+      start = 1;
+      length = numCharacters;
+    }
 
     // Following Presto semantics
     if (start <= 0 || start > numCharacters || length <= 0) {