----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64289/ -----------------------------------------------------------
Review request for kylin. Bugs: KYLIN-3074 https://issues.apache.org/jira/browse/KYLIN-3074 Repository: kylin Description ------- According to ranger recommendations, the following aspects were modified: 1.Cube ACL is removed since v2.2.0, we shuold modify the aclPermission name, no longer cube access. 2.Remove unnecessary requests. For example, get cube and model access entities, it will be recorded in the ranger audit log. 3.Remove unnecessary permission verification. Kylin currently used four permissions(ADMINISTRATION,MANAGEMENT,OPERATION,READ), other permissions do not need to be verified. Every time permission verification will be recorded in the ranger audit log. Diffs ----- server-base/src/main/java/org/apache/kylin/rest/security/AclPermissionFactory.java d66240361 server-base/src/main/java/org/apache/kylin/rest/security/ExternalAclProvider.java 395f44d95 webapp/app/partials/cubes/cubes.html 38c9342bd webapp/app/partials/models/models_tree.html d8aae3487 Diff: https://reviews.apache.org/r/64289/diff/1/ Testing ------- Thanks, pengjianhua