This is an automated email from the ASF dual-hosted git repository.

alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git


The following commit(s) were added to refs/heads/main by this push:
     new 685f5d6c07 fix error message typo (#17570)
685f5d6c07 is described below

commit 685f5d6c07868c8e47d6ffd11fb8134ca6aa405d
Author: Adam Gutglick <a...@spiraldb.com>
AuthorDate: Mon Sep 15 21:50:37 2025 +0100

    fix error message typo (#17570)
---
 datafusion/sql/src/expr/substring.rs    | 3 ++-
 datafusion/sql/tests/sql_integration.rs | 4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/datafusion/sql/src/expr/substring.rs 
b/datafusion/sql/src/expr/substring.rs
index 0cb47a770f..0ff361be0e 100644
--- a/datafusion/sql/src/expr/substring.rs
+++ b/datafusion/sql/src/expr/substring.rs
@@ -79,6 +79,7 @@ impl<S: ContextProvider> SqlToRel<'_, S> {
             }
         }
 
-        not_impl_err!("Substring could not be planned by registered expr 
planner. Hint: enable the `unicode_expressions" )
+        not_impl_err!("Substring could not be planned by registered expr 
planner. \
+                        Hint: Please try with `unicode_expressions` DataFusion 
feature enabled")
     }
 }
diff --git a/datafusion/sql/tests/sql_integration.rs 
b/datafusion/sql/tests/sql_integration.rs
index 6f656e4d8f..9e16ab297e 100644
--- a/datafusion/sql/tests/sql_integration.rs
+++ b/datafusion/sql/tests/sql_integration.rs
@@ -4633,7 +4633,7 @@ fn test_no_substring_registered() {
 
     assert_snapshot!(
         err.strip_backtrace(),
-        @"This feature is not implemented: Substring could not be planned by 
registered expr planner. Hint: enable the `unicode_expressions"
+        @"This feature is not implemented: Substring could not be planned by 
registered expr planner. Hint: Please try with `unicode_expressions` DataFusion 
feature enabled"
     );
 }
 
@@ -4645,7 +4645,7 @@ fn test_no_substring_registered_alt_syntax() {
 
     assert_snapshot!(
         err.strip_backtrace(),
-        @"This feature is not implemented: Substring could not be planned by 
registered expr planner. Hint: enable the `unicode_expressions"
+        @"This feature is not implemented: Substring could not be planned by 
registered expr planner. Hint: Please try with `unicode_expressions` DataFusion 
feature enabled"
     );
 }
 


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@datafusion.apache.org
For additional commands, e-mail: commits-h...@datafusion.apache.org

Reply via email to