From 52cd12a1f9364b0d9315daf80efea3c0f282db1d Mon Sep 17 00:00:00 2001
From: GoGoWen <82132356+GoGoWen@users.noreply.github.com>
Date: Fri, 26 Nov 2021 10:13:53 +0800
Subject: [PATCH] [fix](planner) fix preaggregation reason error (#7205)

this pr is going to Fix #7204.
---
 .../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 ffdfe3caff..c6f4cd3cf0 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();