You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by bi...@apache.org on 2018/02/10 14:24:14 UTC
kylin git commit: KYLIN-3223 fix adjustment based on new ACL check
API added by KYLIN-3239
Repository: kylin
Updated Branches:
refs/heads/master 82f76db0a -> 3fa11b2cf
KYLIN-3223 fix adjustment based on new ACL check API added by KYLIN-3239
Signed-off-by: Billy Liu <bi...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/3fa11b2c
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/3fa11b2c
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/3fa11b2c
Branch: refs/heads/master
Commit: 3fa11b2cffa4e9159b4a5f911dfb1eb1b7c08005
Parents: 82f76db
Author: Seva Ostapenko <se...@anovadata.com>
Authored: Fri Feb 9 11:26:31 2018 -0500
Committer: Billy Liu <bi...@apache.org>
Committed: Sat Feb 10 22:24:00 2018 +0800
----------------------------------------------------------------------
.../java/org/apache/kylin/rest/service/HybridService.java | 10 ++--------
1 file changed, 2 insertions(+), 8 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/kylin/blob/3fa11b2c/server-base/src/main/java/org/apache/kylin/rest/service/HybridService.java
----------------------------------------------------------------------
diff --git a/server-base/src/main/java/org/apache/kylin/rest/service/HybridService.java b/server-base/src/main/java/org/apache/kylin/rest/service/HybridService.java
index e714925..2fb7f84 100644
--- a/server-base/src/main/java/org/apache/kylin/rest/service/HybridService.java
+++ b/server-base/src/main/java/org/apache/kylin/rest/service/HybridService.java
@@ -33,7 +33,6 @@ import org.apache.kylin.storage.hybrid.HybridInstance;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.beans.factory.annotation.Autowired;
-import org.springframework.security.access.AccessDeniedException;
import org.springframework.stereotype.Component;
@Component("hybridService")
@@ -123,13 +122,8 @@ public class HybridService extends BasicService {
if (projectInstance == null) {
continue;
}
- boolean hasReadAccess = false;
- try {
- hasReadAccess = aclEvaluate.hasProjectReadPermission(projectInstance);
- } catch (AccessDeniedException e) {
- //ignore to continue
- }
- if (hasReadAccess) {
+ boolean hasProjectReadAccess = aclEvaluate.hasProjectReadPermission(projectInstance);
+ if (hasProjectReadAccess) {
readableProjects.add(projectInstance);
}
}