Repository: kylin Updated Branches: refs/heads/2.3.x 2a6d69a20 -> 8fa92c70d
KYLIN-3223 fix adjustment based on new ACL check API added by KYLIN-3239 Signed-off-by: Billy Liu <billy...@apache.org> Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/8fa92c70 Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/8fa92c70 Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/8fa92c70 Branch: refs/heads/2.3.x Commit: 8fa92c70dcb9e26a4603f31ce1b4921db7986204 Parents: 2a6d69a Author: Seva Ostapenko <s...@anovadata.com> Authored: Fri Feb 9 11:26:31 2018 -0500 Committer: Billy Liu <billy...@apache.org> Committed: Sat Feb 10 22:24:26 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/8fa92c70/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); } }
