You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by li...@apache.org on 2016/08/22 05:39:55 UTC
[1/2] kylin git commit: KYLIN-1909 Fix cube list ACL bug.
Repository: kylin
Updated Branches:
refs/heads/master a863ae57f -> 4c897dd2e
KYLIN-1909 Fix cube list ACL bug.
Signed-off-by: lidongsjtu <li...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/f41f8c55
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/f41f8c55
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/f41f8c55
Branch: refs/heads/master
Commit: f41f8c55614172134105441f864b0c8a1f55583e
Parents: a863ae5
Author: sanjulian <ju...@hotmail.com>
Authored: Mon Aug 22 12:49:16 2016 +0800
Committer: lidongsjtu <li...@apache.org>
Committed: Mon Aug 22 13:31:04 2016 +0800
----------------------------------------------------------------------
.../kylin/rest/controller/CubeController.java | 23 ++++++++++++++------
.../apache/kylin/rest/service/CubeService.java | 20 -----------------
2 files changed, 16 insertions(+), 27 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/f41f8c55/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 57b0965..ecdab40 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
@@ -20,12 +20,7 @@ package org.apache.kylin.rest.controller;
import java.io.IOException;
import java.net.UnknownHostException;
-import java.util.ArrayList;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
-import java.util.UUID;
+import java.util.*;
import org.apache.commons.lang.StringUtils;
import org.apache.kylin.common.util.JsonUtil;
@@ -98,7 +93,21 @@ public class CubeController extends BasicController {
@RequestMapping(value = "", method = { RequestMethod.GET })
@ResponseBody
public List<CubeInstance> getCubes(@RequestParam(value = "cubeName", required = false) String cubeName, @RequestParam(value = "modelName", required = false) String modelName, @RequestParam(value = "projectName", required = false) String projectName, @RequestParam(value = "limit", required = false) Integer limit, @RequestParam(value = "offset", required = false) Integer offset) {
- return cubeService.getCubes(cubeName, projectName, modelName, limit, offset);
+ List<CubeInstance> cubes;
+ cubes = cubeService.listAllCubes(cubeName, projectName, modelName);
+
+ int climit = (null == limit) ? cubes.size() : limit;
+ int coffset = (null == offset) ? 0 : offset;
+
+ if (cubes.size() <= coffset) {
+ return Collections.emptyList();
+ }
+
+ if ((cubes.size() - coffset) < climit) {
+ return cubes.subList(coffset, cubes.size());
+ }
+
+ return cubes.subList(coffset, coffset + climit);
}
@RequestMapping(value = "validEncodings", method = { RequestMethod.GET })
http://git-wip-us.apache.org/repos/asf/kylin/blob/f41f8c55/server-base/src/main/java/org/apache/kylin/rest/service/CubeService.java
----------------------------------------------------------------------
diff --git a/server-base/src/main/java/org/apache/kylin/rest/service/CubeService.java b/server-base/src/main/java/org/apache/kylin/rest/service/CubeService.java
index cfb4cf8..4cd527c 100644
--- a/server-base/src/main/java/org/apache/kylin/rest/service/CubeService.java
+++ b/server-base/src/main/java/org/apache/kylin/rest/service/CubeService.java
@@ -128,26 +128,6 @@ public class CubeService extends BasicService {
return filterCubes;
}
- @PostFilter(Constant.ACCESS_POST_FILTER_READ)
- public List<CubeInstance> getCubes(final String cubeName, final String projectName, final String modelName, final Integer limit, final Integer offset) {
-
- List<CubeInstance> cubes;
- cubes = listAllCubes(cubeName, projectName, modelName);
-
- int climit = (null == limit) ? cubes.size() : limit;
- int coffset = (null == offset) ? 0 : offset;
-
- if (cubes.size() <= coffset) {
- return Collections.emptyList();
- }
-
- if ((cubes.size() - coffset) < climit) {
- return cubes.subList(coffset, cubes.size());
- }
-
- return cubes.subList(coffset, coffset + climit);
- }
-
@PreAuthorize(Constant.ACCESS_HAS_ROLE_ADMIN + " or hasPermission(#cube, 'ADMINISTRATION') or hasPermission(#cube, 'MANAGEMENT')")
public CubeInstance updateCubeCost(CubeInstance cube, int cost) throws IOException {
[2/2] kylin git commit: KYLIN-1909 Fix test and style
Posted by li...@apache.org.
KYLIN-1909 Fix test and style
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/4c897dd2
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/4c897dd2
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/4c897dd2
Branch: refs/heads/master
Commit: 4c897dd2e8b008c3baaa970d8a0f50d88137026b
Parents: f41f8c5
Author: lidongsjtu <li...@apache.org>
Authored: Mon Aug 22 13:39:29 2016 +0800
Committer: lidongsjtu <li...@apache.org>
Committed: Mon Aug 22 13:39:29 2016 +0800
----------------------------------------------------------------------
.../org/apache/kylin/rest/controller/CubeController.java | 8 +++++++-
.../org/apache/kylin/rest/controller/CubeControllerTest.java | 6 ++++++
.../java/org/apache/kylin/rest/service/CubeServiceTest.java | 5 +----
3 files changed, 14 insertions(+), 5 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/4c897dd2/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 ecdab40..7081d02 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
@@ -20,7 +20,13 @@ package org.apache.kylin.rest.controller;
import java.io.IOException;
import java.net.UnknownHostException;
-import java.util.*;
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.UUID;
import org.apache.commons.lang.StringUtils;
import org.apache.kylin.common.util.JsonUtil;
http://git-wip-us.apache.org/repos/asf/kylin/blob/4c897dd2/server/src/test/java/org/apache/kylin/rest/controller/CubeControllerTest.java
----------------------------------------------------------------------
diff --git a/server/src/test/java/org/apache/kylin/rest/controller/CubeControllerTest.java b/server/src/test/java/org/apache/kylin/rest/controller/CubeControllerTest.java
index f6db8c1..6319899 100644
--- a/server/src/test/java/org/apache/kylin/rest/controller/CubeControllerTest.java
+++ b/server/src/test/java/org/apache/kylin/rest/controller/CubeControllerTest.java
@@ -163,4 +163,10 @@ public class CubeControllerTest extends ServiceTestBase {
Assert.assertTrue(segNumber == newSegNumber + 1);
}
+ @Test
+ public void testGetCubes() {
+ List<CubeInstance> cubes = cubeController.getCubes(null, null, null, 1, 0);
+ Assert.assertTrue(cubes.size() == 1);
+ }
+
}
http://git-wip-us.apache.org/repos/asf/kylin/blob/4c897dd2/server/src/test/java/org/apache/kylin/rest/service/CubeServiceTest.java
----------------------------------------------------------------------
diff --git a/server/src/test/java/org/apache/kylin/rest/service/CubeServiceTest.java b/server/src/test/java/org/apache/kylin/rest/service/CubeServiceTest.java
index a3c983d..f98d6b9 100644
--- a/server/src/test/java/org/apache/kylin/rest/service/CubeServiceTest.java
+++ b/server/src/test/java/org/apache/kylin/rest/service/CubeServiceTest.java
@@ -48,12 +48,9 @@ public class CubeServiceTest extends ServiceTestBase {
Assert.assertNotNull(cubeService.getMetadataManager());
Assert.assertNotNull(cacheService.getOLAPDataSource(ProjectInstance.DEFAULT_PROJECT_NAME));
- List<CubeInstance> cubes = cubeService.getCubes(null, null, null, null, null);
+ List<CubeInstance> cubes = cubeService.listAllCubes(null, null, null);
Assert.assertNotNull(cubes);
CubeInstance cube = cubes.get(0);
cubeService.isCubeDescFreeEditable(cube.getDescriptor());
-
- cubes = cubeService.getCubes(null, null, null, 1, 0);
- Assert.assertTrue(cubes.size() == 1);
}
}