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

lide pushed a commit to branch branch-1.2-lts
in repository https://gitbox.apache.org/repos/asf/doris.git


The following commit(s) were added to refs/heads/branch-1.2-lts by this push:
     new 8ff7689d467 [fix](jdbc_catalog) fix datetime error not throw in jdbc 
catalog (#34975)
8ff7689d467 is described below

commit 8ff7689d4670767db8fecaf3942a1fc140af30d6
Author: Yulei-Yang <yulei.yang0...@gmail.com>
AuthorDate: Thu May 16 19:26:04 2024 +0800

    [fix](jdbc_catalog) fix datetime error not throw in jdbc catalog (#34975)
---
 .../src/main/java/org/apache/doris/analysis/StringLiteral.java       | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git 
a/fe/fe-core/src/main/java/org/apache/doris/analysis/StringLiteral.java 
b/fe/fe-core/src/main/java/org/apache/doris/analysis/StringLiteral.java
index 7690302fbce..c45c6940582 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/analysis/StringLiteral.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/analysis/StringLiteral.java
@@ -266,13 +266,10 @@ public class StringLiteral extends LiteralExpr {
                     break;
             }
         } else if (targetType.isDateType()) {
-            // FE only support 'yyyy-MM-dd hh:mm:ss' && 'yyyy-MM-dd' format
-            // so if FE unchecked cast fail, we also build CastExpr for BE
-            // BE support other format suck as 'yyyyMMdd'...
             try {
                 return convertToDate(targetType);
             } catch (AnalysisException e) {
-                // pass;
+                throw new AnalysisException(e.getMessage());
             }
         } else if (targetType.equals(type)) {
             return this;


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

Reply via email to