Youngwb commented on a change in pull request #3439:
URL: https://github.com/apache/incubator-doris/pull/3439#discussion_r420625860
##########
File path: fe/src/main/java/org/apache/doris/analysis/FunctionCallExpr.java
##########
@@ -605,6 +605,19 @@ public void analyzeImpl(Analyzer analyzer) throws
AnalysisException {
return;
}
+ if
(fnName.getFunction().equalsIgnoreCase("append_trailing_char_if_absent")) {
Review comment:
Yes，I understand there will make this function more complicate, but I
need to avoid two parameters be Non-String type，such as
`append_trailing_char_if_absent(1,2)`, and I don't known how to handle this
case if i don't check here. Maybe there is a better way to handle this case if
you known.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
