> On May 15, 2020, 3:57 a.m., Pradeep Agrawal wrote:
> > Presto plugin was added in ranger-2.0. Will these changes be available for 
> > users who upgrade from ranger-2.0 ? What will be the behaviour with this 
> > change in ranger upgrade case.

the plugin itself is backwards compatible with older Prestos.

note I recently committed an uodate to this plugin "after ranger 2.0" tbis is 
merely a follow up.


- Bolke


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


On May 14, 2020, 8:25 p.m., Bolke de Bruin wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72513/
> -----------------------------------------------------------
> 
> (Updated May 14, 2020, 8:25 p.m.)
> 
> 
> Review request for ranger, Madhan Neethiraj, Mehul Parikh, Pradeep Agrawal, 
> and Ramesh Mani.
> 
> 
> Bugs: https://jira.apache.org/jira/browse/RANGER-2826
>     
> https://issues.apache.org/jira/browse/https://jira.apache.org/jira/browse/RANGER-2826
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> Presto 332/333 are backwards incompatible.
> 
> 
> Diffs
> -----
> 
>   agents-common/src/main/resources/service-defs/ranger-servicedef-presto.json 
> 4d5b79582 
>   
> plugin-presto/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java
>  d4521a392 
>   
> plugin-presto/src/test/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControlTest.java
>  c00d51986 
>   plugin-presto/src/test/resources/presto-policies.json 28eabf2d6 
>   pom.xml ebce7c9f0 
>   
> ranger-presto-plugin-shim/src/main/java/org/apache/ranger/authorization/presto/authorizer/RangerSystemAccessControl.java
>  bfb3a5961 
> 
> 
> Diff: https://reviews.apache.org/r/72513/diff/1/
> 
> 
> Testing
> -------
> 
> Unit tests updated. Production.
> 
> 
> Thanks,
> 
> Bolke de Bruin
> 
>

Reply via email to