You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by ya...@apache.org on 2021/11/17 09:20:36 UTC

[kylin] branch kylin3-hadoop3 updated (de748ea -> 7372ade)

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

yaqian pushed a change to branch kylin3-hadoop3
in repository https://gitbox.apache.org/repos/asf/kylin.git.


    from de748ea  [maven-release-plugin] prepare release kylin-3.1.3
     new efee7f7  Fix the bug of CubeDesc can't be update
     new 7372ade  Revert "[KYLIN-5032] Not allow change project for a cubeInstance."

The 2 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.


Summary of changes:
 .../java/org/apache/kylin/rest/controller/CubeController.java     | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

[kylin] 02/02: Revert "[KYLIN-5032] Not allow change project for a cubeInstance."

Posted by ya...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

yaqian pushed a commit to branch kylin3-hadoop3
in repository https://gitbox.apache.org/repos/asf/kylin.git

commit 7372ade192ff9feb2a3855488a7164a10d2dc484
Author: yaqian.zhang <59...@qq.com>
AuthorDate: Mon Nov 15 15:42:36 2021 +0800

    Revert "[KYLIN-5032] Not allow change project for a cubeInstance."
    
    This reverts commit f8f52aff
---
 .../main/java/org/apache/kylin/rest/controller/CubeController.java  | 6 ------
 1 file changed, 6 deletions(-)

diff --git a/server-base/src/main/java/org/apache/kylin/rest/controller/CubeController.java b/server-base/src/main/java/org/apache/kylin/rest/controller/CubeController.java
index 677e723..589db07 100644
--- a/server-base/src/main/java/org/apache/kylin/rest/controller/CubeController.java
+++ b/server-base/src/main/java/org/apache/kylin/rest/controller/CubeController.java
@@ -861,12 +861,6 @@ public class CubeController extends BasicController {
                 return cubeRequest;
             }
 
-            if (!cube.getProject().equalsIgnoreCase(projectName)) {
-                String error = "Not allow change cube project!";
-                updateRequest(cubeRequest, false, error);
-                return cubeRequest;
-            }
-
             validateColumnFamily(desc);
 
             // check build engine for stream cube

[kylin] 01/02: Fix the bug of CubeDesc can't be update

Posted by ya...@apache.org.
This is an automated email from the ASF dual-hosted git repository.

yaqian pushed a commit to branch kylin3-hadoop3
in repository https://gitbox.apache.org/repos/asf/kylin.git

commit efee7f7cec1f08e6274c01037605064937c927f0
Author: yaqian.zhang <59...@qq.com>
AuthorDate: Mon Nov 8 17:51:03 2021 +0800

    Fix the bug of CubeDesc can't be update
---
 .../src/main/java/org/apache/kylin/rest/controller/CubeController.java  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/server-base/src/main/java/org/apache/kylin/rest/controller/CubeController.java b/server-base/src/main/java/org/apache/kylin/rest/controller/CubeController.java
index 2124404..677e723 100644
--- a/server-base/src/main/java/org/apache/kylin/rest/controller/CubeController.java
+++ b/server-base/src/main/java/org/apache/kylin/rest/controller/CubeController.java
@@ -870,7 +870,7 @@ public class CubeController extends BasicController {
             validateColumnFamily(desc);
 
             // check build engine for stream cube
-            if (desc.isStreamingCube()) {
+            if (cube.getDescriptor().isStreamingCube()) {
                 if (desc.getEngineType() != IEngineAware.ID_MR_V2) {
                     logger.info("streaming cube just supports MR engine");
                     throw new BadRequestException("Invalid Engine type, Streaming cube just supports MapReduce engine");