You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by ni...@apache.org on 2019/11/11 08:15:55 UTC

[kylin] branch 2.6.x updated: Revert "KYLIN-3999, make kylin.query.enable-dynamic-column default true"

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

nic pushed a commit to branch 2.6.x
in repository https://gitbox.apache.org/repos/asf/kylin.git


The following commit(s) were added to refs/heads/2.6.x by this push:
     new 9bb057f  Revert "KYLIN-3999, make kylin.query.enable-dynamic-column default true"
9bb057f is described below

commit 9bb057f03b6964714edd2c3401ae61f6e1b0ba40
Author: rupengwang <wa...@livec.cn>
AuthorDate: Mon Nov 11 15:40:39 2019 +0800

    Revert "KYLIN-3999, make kylin.query.enable-dynamic-column default true"
    
    This reverts commit 5b1d0a5c
---
 core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java b/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java
index aa8626f..92f3cb5 100644
--- a/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java
+++ b/core-common/src/main/java/org/apache/kylin/common/KylinConfigBase.java
@@ -1446,7 +1446,7 @@ abstract public class KylinConfigBase implements Serializable {
     // check KYLIN-3358, need deploy coprocessor if enabled
     // finally should be deprecated
     public boolean isDynamicColumnEnabled() {
-        return Boolean.parseBoolean(getOptional("kylin.query.enable-dynamic-column", TRUE));
+        return Boolean.parseBoolean(getOptional("kylin.query.enable-dynamic-column", FALSE));
     }
 
     //check KYLIN-1684, in most cases keep the default value