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

[fix](Nereids) count should not accept complex and json type (#25354)

Browse Source
This commit is contained in:
morrySnow
2023-10-16 11:08:35 +08:00
committed by GitHub
parent dfc7d04626
commit 4c57c31c5c
4 changed files with 184 additions and 178 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
fe/fe-core/src/main/java/org/apache/doris/nereids/trees/expressions/functions/agg/Count.java
View File

@ -66,7 +66,7 @@ public class Count extends AggregateFunction
public boolean isCountStar() {
return isStar
|| children.size() == 0
|| children.isEmpty()
|| (children.size() == 1 && child(0) instanceof Literal);
}
@ -82,8 +82,11 @@ public class Count extends AggregateFunction
public void checkLegalityAfterRewrite() {
// after rewrite, count(distinct bitmap_column) should be rewritten to bitmap_union_count(bitmap_column)
for (Expression argument : getArguments()) {
if (argument.getDataType().isOnlyMetricType()) {
throw new AnalysisException(Type.OnlyMetricTypeErrorMsg);
if (argument.getDataType().isObjectType()) {
throw new AnalysisException(Type.OnlyObjectTypeErrorMsg);
}
if (distinct && argument.getDataType().isComplexType()) {
throw new AnalysisException("COUNT DISTINCT could not process complex type " + this.toSql());
}
}
}