You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by sh...@apache.org on 2018/01/04 08:12:17 UTC

[2/2] kylin git commit: KYLIN-1925, forbid cloning cubes across projects

KYLIN-1925, forbid cloning cubes across projects

Signed-off-by: shaofengshi <sh...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/e44f95e6
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/e44f95e6
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/e44f95e6

Branch: refs/heads/master
Commit: e44f95e632bb11dad6c2df16f8e0398b09e3e8c3
Parents: 7f3fdbe
Author: etherge <et...@163.com>
Authored: Thu Jan 4 10:13:49 2018 +0800
Committer: shaofengshi <sh...@apache.org>
Committed: Thu Jan 4 11:25:35 2018 +0800

----------------------------------------------------------------------
 .../java/org/apache/kylin/rest/controller/CubeController.java    | 4 ++++
 1 file changed, 4 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/e44f95e6/server-base/src/main/java/org/apache/kylin/rest/controller/CubeController.java
----------------------------------------------------------------------
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 e823ebe..a23c9b7 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
@@ -507,6 +507,10 @@ public class CubeController extends BasicController {
         if (project == null) {
             throw new BadRequestException("Project " + projectName + " doesn't exist");
         }
+        // KYLIN-1925, forbid cloning cross projects
+        if (!project.getName().equals(cube.getProject())) {
+            throw new BadRequestException("Cloning cubes across projects is not supported.");
+        }
 
         CubeDesc cubeDesc = cube.getDescriptor();
         CubeDesc newCubeDesc = CubeDesc.getCopyOf(cubeDesc);