You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ranger.apache.org by Abhay Kulkarni <ak...@hortonworks.com> on 2020/04/30 21:29:51 UTC

Re: Review Request 72455: RANGER-2814: RangerServiceResource signature should not include service-specific information

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72455/
-----------------------------------------------------------

(Updated April 30, 2020, 9:29 p.m.)


Review request for ranger and Madhan Neethiraj.


Changes
-------

Updated with Apache JIRA information


Summary (updated)
-----------------

RANGER-2814: RangerServiceResource signature should not include service-specific information


Bugs: RANGER-2814
    https://issues.apache.org/jira/browse/RANGER-2814


Repository: ranger


Description (updated)
-------

RangerServiceResource objects represent a resource within scope of Ranger service in Ranger world. For efficient searching, resource contains a signature (which is a digest of its contents) which is computed at its creation as well as when it is updated. 

As the search for a resource is always carried out in the scope of Ranger service, there is no need to include service name (or service id) in its signature. In fact,  having service name included in the signature makes upgrade/migration of tagged service resources  very difficult.

Along with removing service-name from the signature of.a service resource, a patch to migrate existing service resources to new signature is also needed.


Diffs
-----

  agents-common/src/main/java/org/apache/ranger/plugin/store/RangerServiceResourceSignature.java eb9a9d240 
  security-admin/db/mysql/optimized/current/ranger_core_db_mysql.sql 320ab6d30 
  security-admin/db/oracle/optimized/current/ranger_core_db_oracle.sql fbf91d861 
  security-admin/db/postgres/optimized/current/ranger_core_db_postgres.sql 74c7ffc76 
  security-admin/db/sqlanywhere/optimized/current/ranger_core_db_sqlanywhere.sql 4715584d7 
  security-admin/db/sqlserver/optimized/current/ranger_core_db_sqlserver.sql 1a966da0c 
  security-admin/src/main/java/org/apache/ranger/patch/PatchForMigratingRangerServiceResource_J10037.java PRE-CREATION 


Diff: https://reviews.apache.org/r/72455/diff/2/


Testing
-------

Tested with ~100,000 service resources. Took about 3 minutes.


Thanks,

Abhay Kulkarni