-----------------------------------------------------------
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

Reply via email to