[kylin] 02/02: KYLIN-5250 work round for no hack aggregation group

[xxyu]

This is an automated email from the ASF dual-hosted git repository.

xxyu pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/kylin.git

commit 04f1361ff5216bf53662fb9afa24a135341c7e19
Author: Mukvin <boyboys...@163.com>
AuthorDate: Fri Sep 2 14:58:01 2022 +0800

    KYLIN-5250 work round for no hack aggregation group
---
 .../src/main/java/org/apache/kylin/cube/RawQueryLastHacker.java    | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git 
a/core-cube/src/main/java/org/apache/kylin/cube/RawQueryLastHacker.java 
b/core-cube/src/main/java/org/apache/kylin/cube/RawQueryLastHacker.java
index fba5fe83d5..89ef4435a7 100644
--- a/core-cube/src/main/java/org/apache/kylin/cube/RawQueryLastHacker.java
+++ b/core-cube/src/main/java/org/apache/kylin/cube/RawQueryLastHacker.java
@@ -35,10 +35,9 @@ public class RawQueryLastHacker {
     private static final Logger logger = 
LoggerFactory.getLogger(RawQueryLastHacker.class);
 
     public static void hackNoAggregations(SQLDigest sqlDigest, CubeDesc 
cubeDesc, TupleInfo tupleInfo) {
-        if (!sqlDigest.isRawQuery ||
-                BackdoorToggles.getDisabledRawQueryLastHacker() ||
-                KylinConfig.getInstanceFromEnv().isEnabledNoAggQuery()) {
-            if (KylinConfig.getInstanceFromEnv().isEnabledNoAggQuery()) {
+        boolean isEnabledNoAggQuery = 
KylinConfig.getInstanceFromEnv().isEnabledNoAggQuery();
+        if (!sqlDigest.isRawQuery || 
BackdoorToggles.getDisabledRawQueryLastHacker() || isEnabledNoAggQuery) {
+            if (isEnabledNoAggQuery) {
                 logger.info("No hack for aggregation sql by 
kylin.query.enable-no-aggregate-query which is 'true'.");
             }
             return;