Add comment about why it is ok to return boolean with catchall match

apache · Aug 8, 2024 · ead4a7c · ead4a7c
1 parent 268efcd
commit ead4a7c
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/datafusion/functions/src/regex/regexplike.rs b/datafusion/functions/src/regex/regexplike.rs
@@ -76,6 +76,8 @@ impl ScalarUDFImpl for RegexpLikeFunc {
 
         Ok(match &arg_types[0] {
             Null => Null,
+            // Type coercion is done by DataFusion based on signature, so if we
+            // get here, the first argument is always a string
             _ => Boolean,
         })
     }