Apache
/
druid
mirror of https://github.com/apache/druid.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix druid sql group by queries returning complex aggregation type (#8099) 

* Fix druid sql group by queries returning complex aggregation type

* Remove unnecessary check
This commit is contained in:
Samarth Jain 2019-07-19 13:52:14 -07:00 committed by Gian Merlino
parent 01f3da6fda
commit ceb3a891bb
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
sql/src/main/java/org/apache/druid/sql/calcite/rel/Projection.java
View File

@ -28,6 +28,7 @@ import org.apache.druid.math.expr.ExprType;
import org.apache.druid.query.aggregation.PostAggregator;
import org.apache.druid.query.aggregation.post.ExpressionPostAggregator;
import org.apache.druid.segment.VirtualColumn;
import org.apache.druid.segment.column.ValueType;
import org.apache.druid.sql.calcite.expression.DruidExpression;
import org.apache.druid.sql.calcite.expression.Expressions;
import org.apache.druid.sql.calcite.planner.Calcites;
@ -193,11 +194,14 @@ public class Projection
return false;
}
// Check if a cast is necessary.
final ExprType toExprType = Expressions.exprTypeForValueType(
aggregateRowSignature.getColumnType(expression.getDirectColumn())
);
// We don't really have a way to cast complex type. So might as well not do anything and return.
final ValueType columnValueType = aggregateRowSignature.getColumnType(expression.getDirectColumn());
if (columnValueType == ValueType.COMPLEX) {
return true;
}
// Check if a cast is necessary.
final ExprType toExprType = Expressions.exprTypeForValueType(columnValueType);
final ExprType fromExprType = Expressions.exprTypeForValueType(
Calcites.getValueTypeForSqlTypeName(rexNode.getType().getSqlTypeName())
);