You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by mo...@apache.org on 2021/11/26 02:14:07 UTC
[incubator-doris] branch master updated: [fix](planner) fix preaggregation reason error (#7205)
This is an automated email from the ASF dual-hosted git repository.
morningman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-doris.git
The following commit(s) were added to refs/heads/master by this push:
new 52cd12a [fix](planner) fix preaggregation reason error (#7205)
52cd12a is described below
commit 52cd12a1f9364b0d9315daf80efea3c0f282db1d
Author: GoGoWen <82...@users.noreply.github.com>
AuthorDate: Fri Nov 26 10:13:53 2021 +0800
[fix](planner) fix preaggregation reason error (#7205)
this pr is going to Fix #7204.
---
.../src/main/java/org/apache/doris/planner/SingleNodePlanner.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java b/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java
index ffdfe3c..c6f4cd3 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/planner/SingleNodePlanner.java
@@ -550,8 +550,8 @@ public class SingleNodePlanner {
continue;
}
if (col.isKey()) {
- if (aggExpr.getFnName().getFunction().equalsIgnoreCase("MAX")
- && aggExpr.getFnName().getFunction().equalsIgnoreCase("MIN")) {
+ if ((!aggExpr.getFnName().getFunction().equalsIgnoreCase("MAX"))
+ && (!aggExpr.getFnName().getFunction().equalsIgnoreCase("MIN"))) {
returnColumnValidate = false;
turnOffReason = "the type of agg on StorageEngine's Key column should only be MAX or MIN."
+ "agg expr: " + aggExpr.toSql();
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org