Re: Review Request 72551: RANGER-2834: Upgrade Ranger to support Elasticsearch 7.6.0

2020-05-31 Thread bhavik patel

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


Ship it!




Ship It!

- bhavik patel


On May 27, 2020, 2:46 p.m., Pradeep Agrawal wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72551/
> ---
> 
> (Updated May 27, 2020, 2:46 p.m.)
> 
> 
> Review request for ranger, Andrew Charneski, Ankita Sinha, Bolke de Bruin, 
> Don Bosco Durai, bhavik patel, Colm O hEigeartaigh, Gautam Borad, Jayendra 
> Parab, Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Nikhil P, Nitin 
> Galave, pengjianhua, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, 
> Velmurugan Periasamy, Qiang Zhang, and Barna Zsombor Klara.
> 
> 
> Bugs: RANGER-2834
> https://issues.apache.org/jira/browse/RANGER-2834
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> **Problem Statement:** Ranger currently supports elastic search version 6.2.2 
> which is around 2 year old.
> 
> **Proposed solution:** Here I am suggesting to upgrade to 7.6.0 version.
> 
> 
> Diffs
> -
> 
>   agents-audit/pom.xml 12d621f02 
>   
> agents-audit/src/main/java/org/apache/ranger/audit/destination/ElasticSearchAuditDestination.java
>  dc00fecfe 
>   
> plugin-elasticsearch/src/main/java/org/apache/ranger/authorization/elasticsearch/authorizer/RangerElasticsearchAuthorizer.java
>  bd0585c8a 
>   
> plugin-elasticsearch/src/main/java/org/apache/ranger/services/elasticsearch/RangerServiceElasticsearch.java
>  ed1c98e5c 
>   
> plugin-elasticsearch/src/main/java/org/apache/ranger/services/elasticsearch/client/ElasticsearchClient.java
>  d5c170d47 
>   pom.xml 5605a1701 
>   
> ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/authorizer/RangerElasticsearchAuthorizer.java
>  8f33cb99b 
>   
> ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/RangerElasticsearchPlugin.java
>  faeac5234 
>   
> ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/action/filter/RangerSecurityActionFilter.java
>  60baaea62 
>   
> ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/rest/filter/RangerSecurityRestFilter.java
>  117356a06 
>   
> ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/utils/RequestUtils.java
>  8a04b06d5 
>   
> security-admin/src/main/java/org/apache/ranger/elasticsearch/ElasticSearchAccessAuditsService.java
>  c88726fbb 
>   
> security-admin/src/main/java/org/apache/ranger/elasticsearch/ElasticSearchUtil.java
>  32a45838f 
> 
> 
> Diff: https://reviews.apache.org/r/72551/diff/1/
> 
> 
> Testing
> ---
> 
> Tested ranger audits on elastic search 7.6.0 version
> 
> 
> Prerequisite: For testing of this patch RANGER-2837(RR: 72550) changes are 
> also required.
> 
> 
> Thanks,
> 
> Pradeep Agrawal
> 
>



Review Request 72551: RANGER-2834: Upgrade Ranger to support Elasticsearch 7.6.0

2020-05-27 Thread Pradeep Agrawal

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

Review request for ranger, Andrew Charneski, Ankita Sinha, Bolke de Bruin, Don 
Bosco Durai, bhavik patel, Colm O hEigeartaigh, Gautam Borad, Jayendra Parab, 
Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Nikhil P, Nitin Galave, 
pengjianhua, Ramesh Mani, Selvamohan Neethiraj, Sailaja Polavarapu, Velmurugan 
Periasamy, Qiang Zhang, and Barna Zsombor Klara.


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


Repository: ranger


Description
---

**Problem Statement:** Ranger currently supports elastic search version 6.2.2 
which is around 2 year old.

**Proposed solution:** Here I am suggesting to upgrade to 7.6.0 version.


Diffs
-

  agents-audit/pom.xml 12d621f02 
  
agents-audit/src/main/java/org/apache/ranger/audit/destination/ElasticSearchAuditDestination.java
 dc00fecfe 
  
plugin-elasticsearch/src/main/java/org/apache/ranger/authorization/elasticsearch/authorizer/RangerElasticsearchAuthorizer.java
 bd0585c8a 
  
plugin-elasticsearch/src/main/java/org/apache/ranger/services/elasticsearch/RangerServiceElasticsearch.java
 ed1c98e5c 
  
plugin-elasticsearch/src/main/java/org/apache/ranger/services/elasticsearch/client/ElasticsearchClient.java
 d5c170d47 
  pom.xml 5605a1701 
  
ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/authorizer/RangerElasticsearchAuthorizer.java
 8f33cb99b 
  
ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/RangerElasticsearchPlugin.java
 faeac5234 
  
ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/action/filter/RangerSecurityActionFilter.java
 60baaea62 
  
ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/rest/filter/RangerSecurityRestFilter.java
 117356a06 
  
ranger-elasticsearch-plugin-shim/src/main/java/org/apache/ranger/authorization/elasticsearch/plugin/utils/RequestUtils.java
 8a04b06d5 
  
security-admin/src/main/java/org/apache/ranger/elasticsearch/ElasticSearchAccessAuditsService.java
 c88726fbb 
  
security-admin/src/main/java/org/apache/ranger/elasticsearch/ElasticSearchUtil.java
 32a45838f 


Diff: https://reviews.apache.org/r/72551/diff/1/


Testing
---

Tested ranger audits on elastic search 7.6.0 version


Prerequisite: For testing of this patch RANGER-2837(RR: 72550) changes are also 
required.


Thanks,

Pradeep Agrawal