> On May 16, 2020, 5:47 a.m., Pradeep Agrawal wrote:
> > security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java
> > Lines 4703 (patched)
> > <https://reviews.apache.org/r/72517/diff/1/?file=2231875#file2231875line4703>
> >
> >     Can you add some comment above about why the index should always be 1 
> > here.

It should be 0. We just wanted to use the first element and get the serviceId 
alone and use the Ranger admin policy cache data instead of going to ranger db. 
Currently when there are many policies, export results in performance on ranger 
admin.


- Ramesh


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


On May 16, 2020, 5:26 a.m., Ramesh Mani wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72517/
> -----------------------------------------------------------
> 
> (Updated May 16, 2020, 5:26 a.m.)
> 
> 
> Review request for ranger, Don Bosco Durai, Gautam Borad, Abhay Kulkarni, 
> Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, Selvamohan Neethiraj, 
> Sailaja Polavarapu, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-2828
>     https://issues.apache.org/jira/browse/RANGER-2828
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> RANGER-2828:RangerExportPolicy with resource filter fails to fetch policies
> 
> 
> Diffs
> -----
> 
>   security-admin/src/main/java/org/apache/ranger/biz/ServiceDBStore.java 
> 857a597 
> 
> 
> Diff: https://reviews.apache.org/r/72517/diff/1/
> 
> 
> Testing
> -------
> 
> Verified in local vm with curl
> curl -k -u admin:admin --verbose -X GET 
> 'http://172.25.34.170:6080/service/plugins/policies/exportJson?serviceName=c349_hive&polResource=SZ_SL&resource:database=SZ_SL&serviceType=hive&resourceMatchScope=self_or_ancestor&resourceMatch=full'
>  curl -k -u admin:admin --verbose -X GET 
> 'http://172.25.34.170:6080/service/plugins/policies/exportJson?serviceName=c349_hive&polResource=SZ_SL&resource:database=SZ_SL&serviceType=hive&resourceMatchScope=self_or_ancestor'
> 
> 
> Thanks,
> 
> Ramesh Mani
> 
>

Reply via email to