Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-02-07 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/#review225163 --- Ship it! Ship It! - Madhan Neethiraj On Feb. 7, 2023, 5:34 p

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-02-07 Thread Ramachandran Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/ --- (Updated Feb. 7, 2023, 5:34 p.m.) Review request for ranger, Don Bosco Durai, K

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-02-07 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/#review225162 --- intg/src/main/java/org/apache/ranger/RangerClient.java Lines 291

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-02-07 Thread Ramachandran Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/ --- (Updated Feb. 7, 2023, 1:40 p.m.) Review request for ranger, Don Bosco Durai, K

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-02-02 Thread Ramachandran Krishnan
> On Jan. 31, 2023, 6:42 p.m., Abhishek Kumar wrote: > > Missed to mention we also have ranger python client, please take a look at > > changes required here: > > https://github.com/apache/ranger/blob/master/intg/src/main/python/apache_ranger/client/ranger_client.py madhan I think that we hav

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-31 Thread Ramachandran Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/ --- (Updated Feb. 1, 2023, 3:58 a.m.) Review request for ranger, Don Bosco Durai, K

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-31 Thread Abhishek Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/#review225143 --- Missed to mention we also have ranger python client, please take a

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-26 Thread Ramachandran Krishnan
> On Jan. 24, 2023, 9:06 p.m., Abhishek Kumar wrote: > > security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.java > > Line 464 (original), 464 (patched) > > > > > > Client side code needs update: > >

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-25 Thread Ramachandran Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/ --- (Updated Jan. 25, 2023, 2:49 p.m.) Review request for ranger, Don Bosco Durai,

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-24 Thread Abhishek Kumar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/#review225128 --- security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-24 Thread Abhishek Kumar
> On Jan. 20, 2023, 5:56 p.m., Madhan Neethiraj wrote: > > security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.java > > Line 471 (original), 471 (patched) > > > > > > Multiple policies within a service c

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-22 Thread Ramachandran Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/ --- (Updated Jan. 23, 2023, 4:41 a.m.) Review request for ranger, Don Bosco Durai,

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-21 Thread Ramachandran Krishnan
> On Jan. 20, 2023, 5:56 p.m., Madhan Neethiraj wrote: > > security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.java > > Line 471 (original), 471 (patched) > > > > > > Multiple policies within a service c

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-21 Thread Madhan Neethiraj
> On Jan. 20, 2023, 5:56 p.m., Madhan Neethiraj wrote: > > security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.java > > Line 471 (original), 471 (patched) > > > > > > Multiple policies within a service c

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-20 Thread Ramachandran Krishnan
> On Jan. 20, 2023, 5:56 p.m., Madhan Neethiraj wrote: > > security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.java > > Line 471 (original), 471 (patched) > > > > > > Multiple policies within a service c

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-20 Thread Madhan Neethiraj
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/#review225110 --- security-admin/src/main/java/org/apache/ranger/rest/PublicAPIsv2.

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-20 Thread Ramachandran Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/ --- (Updated Jan. 20, 2023, 3:22 p.m.) Review request for ranger, Don Bosco Durai,

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-20 Thread Ramachandran Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/ --- (Updated Jan. 20, 2023, 3:18 p.m.) Review request for ranger, Don Bosco Durai,

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2023-01-03 Thread Ramachandran Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/ --- (Updated Jan. 4, 2023, 7:12 a.m.) Review request for ranger, Don Bosco Durai, K

Re: Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2022-12-12 Thread Ramachandran Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/ --- (Updated Dec. 12, 2022, 1:27 p.m.) Review request for ranger, Don Bosco Durai,

Review Request 74251: RANGER-4012:getPolicyByName searches policy by serviceName, policyName simply by traverse all policies in RangerServicePoliciesCache instead of DB

2022-12-12 Thread Ramachandran Krishnan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74251/ --- Review request for ranger, Don Bosco Durai, Abhay Kulkarni, Madhan Neethiraj, Me