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/11/28 02:09:19 UTC
[kylin] branch kylin5 updated: KYLIN-5304 Query cache enable should check both 'KylinConfig' and 'BackdoorToggles'
This is an automated email from the ASF dual-hosted git repository.
xxyu pushed a commit to branch kylin5
in repository https://gitbox.apache.org/repos/asf/kylin.git
The following commit(s) were added to refs/heads/kylin5 by this push:
new 140f2792b9 KYLIN-5304 Query cache enable should check both 'KylinConfig' and 'BackdoorToggles'
140f2792b9 is described below
commit 140f2792b9bf7ccfb4f560a52f805434800af02b
Author: zhennzhang <zh...@ebay.com>
AuthorDate: Fri Nov 25 17:27:06 2022 +0800
KYLIN-5304 Query cache enable should check both 'KylinConfig' and 'BackdoorToggles'
---
.../src/main/java/org/apache/kylin/rest/service/QueryService.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/query-service/src/main/java/org/apache/kylin/rest/service/QueryService.java b/src/query-service/src/main/java/org/apache/kylin/rest/service/QueryService.java
index 5438ee1ef9..c3cbe877cb 100644
--- a/src/query-service/src/main/java/org/apache/kylin/rest/service/QueryService.java
+++ b/src/query-service/src/main/java/org/apache/kylin/rest/service/QueryService.java
@@ -604,7 +604,7 @@ public class QueryService extends BasicService implements CacheSignatureQuerySup
sqlResponse = QueryUtils.handleTempStatement(sqlRequest, kylinConfig);
// search cache
- if (sqlResponse == null && kylinConfig.isQueryCacheEnabled() && !sqlRequest.isForcedToPushDown()) {
+ if (sqlResponse == null && isQueryCacheEnabled(kylinConfig) && !sqlRequest.isForcedToPushDown()) {
sqlResponse = searchCache(sqlRequest, kylinConfig);
}