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

[fix](Nereids) divide operator return type is not same with lagecy planner (#15707)

Browse Source
This commit is contained in:
morrySnow
2023-01-09 14:50:24 +08:00
committed by GitHub
parent 93b941baeb
commit 4f2bea86ee
1 changed files with 13 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@ -20,6 +20,9 @@ package org.apache.doris.nereids.trees.expressions;
import org.apache.doris.analysis.ArithmeticExpr.Operator;
import org.apache.doris.nereids.exceptions.UnboundException;
import org.apache.doris.nereids.trees.expressions.visitor.ExpressionVisitor;
import org.apache.doris.nereids.types.DataType;
import org.apache.doris.nereids.types.DecimalV2Type;
import org.apache.doris.nereids.types.DoubleType;
import org.apache.doris.nereids.types.coercion.AbstractDataType;
import org.apache.doris.nereids.types.coercion.NumericType;
@ -52,6 +55,16 @@ public class Divide extends BinaryArithmetic {
return NumericType.INSTANCE;
}
@Override
public DataType getDataType() throws UnboundException {
DataType commonType = super.getDataType();
if (commonType.isDecimalType()) {
return DecimalV2Type.SYSTEM_DEFAULT;
} else {
return DoubleType.INSTANCE;
}
}
// Divide is implemented as a scalar function which return type is always nullable.
@Override
public boolean nullable() throws UnboundException {