You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by xx...@apache.org on 2022/09/02 07:24:05 UTC
[kylin] 02/02: KYLIN-5250 work round for no hack aggregation group
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 <bo...@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;