database/doris
2
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

[fix](nereids) cast left child of TimestampArithmetic to wrong type in BindFunction (#12423)

Browse Source
This commit is contained in:
Kikyou1997
2022-09-07 20:32:47 +08:00
committed by GitHub
parent c2808de867
commit bdbce77227
2 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
fe/fe-core/src/main/java/org/apache/doris/nereids/rules/analysis/BindFunction.java
View File

@ -175,7 +175,7 @@ public class BindFunction implements AnalysisRuleFactory {
} catch (Exception e) {
// ignore
}
if (!left.getDataType().isDateType() && arithmetic.getTimeUnit().isDateTimeUnit()) {
if (!left.getDataType().isDateType() && !arithmetic.getTimeUnit().isDateTimeUnit()) {
left = arithmetic.left().castTo(DateType.INSTANCE);
}
}

1
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/TimestampArithmetic.java
View File

@ -36,6 +36,7 @@ import java.util.List;
* Describes the addition and subtraction of time units from timestamps.
* Arithmetic expressions on timestamps are syntactic sugar.
* They are executed as function call exprs in the BE.
* Example: '1996-01-01' + INTERVAL '3' month;
* TODO: we need to rethink this, and maybe need to add a new type of Interval then implement IntervalLiteral as others
*/
public class TimestampArithmetic extends Expression implements BinaryExpression {