This is an automated email from the ASF dual-hosted git repository. dataroaring pushed a commit to branch branch-3.0 in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-3.0 by this push: new 0e33923c3f0 branch-3.0: [fix](Nereids) fix initcap fe fold constant #49061 (#49583) 0e33923c3f0 is described below commit 0e33923c3f0e89e0e54da542aecf1550c33fc3f1 Author: LiBinfeng <libinf...@selectdb.com> AuthorDate: Sun Mar 30 10:51:04 2025 +0800 branch-3.0: [fix](Nereids) fix initcap fe fold constant #49061 (#49583) Cherry-picked from https://github.com/apache/doris/pull/49061 case of initcap has been added at: (https://github.com/apache/doris/pull/49346) --- .../expressions/functions/executable/StringArithmetic.java | 11 ++++------- 1 file changed, 4 insertions(+), 7 deletions(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/executable/StringArithmetic.java b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/executable/StringArithmetic.java index 411f070d24d..e714ab6a87d 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/executable/StringArithmetic.java +++ b/fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/executable/StringArithmetic.java @@ -420,12 +420,9 @@ public class StringArithmetic { return new IntegerLiteral(first.getValue().codePointCount(0, first.getValue().length())); } - private static boolean isSeparator(char c) { - if (".$|()[{^?*+\\".indexOf(c) == -1) { - return false; - } else { - return true; - } + private static boolean isAlphabetic(char c) { + Pattern pattern = Pattern.compile("\\p{Alnum}"); + return pattern.matcher(String.valueOf(c)).find(); } /** @@ -437,7 +434,7 @@ public class StringArithmetic { boolean capitalizeNext = true; for (char c : first.getValue().toCharArray()) { - if (Character.isWhitespace(c) || isSeparator(c)) { + if (Character.isWhitespace(c) || !isAlphabetic(c)) { result.append(c); capitalizeNext = true; // Next character should be capitalized } else if (capitalizeNext) { --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org