Re: Review Request 73338: RANGER-3275 : Need to update solr-config.xml in the ranger-audits collection config-set

2021-05-10 Thread Mehul Parikh

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


Ship it!




Ship It!

- Mehul Parikh


On May 6, 2021, 4:43 p.m., Vishal Suvagia wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73338/
> ---
> 
> (Updated May 6, 2021, 4:43 p.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Dhaval Shah, Dineshkumar Yadav, 
> Gautam Borad, Jayendra Parab, Kishor Gollapalliwar, Abhay Kulkarni, Madhan 
> Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, Sailaja Polavarapu, 
> and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3275
> https://issues.apache.org/jira/browse/RANGER-3275
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> The solrconfig.xml in the config-set for Ranger needs to be updated in 
> correct order to use the ttl configuration properly, without which the 
> documents do not contain the ttl and will not expire or purge out even after 
> the ttl is set.
> 
> 
> Diffs
> -
> 
>   security-admin/contrib/solr_for_audit_setup/conf/solrconfig.xml 
> 2216f665fd197e066585fe527645b218ab25a221 
> 
> 
> Diff: https://reviews.apache.org/r/73338/diff/1/
> 
> 
> Testing
> ---
> 
> Tested the changes locally, ttl is getting applied to the created documents.
> 
> 
> Thanks,
> 
> Vishal Suvagia
> 
>



Re: Review Request 73338: RANGER-3275 : Need to update solr-config.xml in the ranger-audits collection config-set

2021-05-06 Thread Ramesh Mani

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


Ship it!




Ship It!

- Ramesh Mani


On May 6, 2021, 4:43 p.m., Vishal Suvagia wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73338/
> ---
> 
> (Updated May 6, 2021, 4:43 p.m.)
> 
> 
> Review request for ranger, Ankita Sinha, Dhaval Shah, Dineshkumar Yadav, 
> Gautam Borad, Jayendra Parab, Kishor Gollapalliwar, Abhay Kulkarni, Madhan 
> Neethiraj, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, Sailaja Polavarapu, 
> and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3275
> https://issues.apache.org/jira/browse/RANGER-3275
> 
> 
> Repository: ranger
> 
> 
> Description
> ---
> 
> The solrconfig.xml in the config-set for Ranger needs to be updated in 
> correct order to use the ttl configuration properly, without which the 
> documents do not contain the ttl and will not expire or purge out even after 
> the ttl is set.
> 
> 
> Diffs
> -
> 
>   security-admin/contrib/solr_for_audit_setup/conf/solrconfig.xml 
> 2216f665fd197e066585fe527645b218ab25a221 
> 
> 
> Diff: https://reviews.apache.org/r/73338/diff/1/
> 
> 
> Testing
> ---
> 
> Tested the changes locally, ttl is getting applied to the created documents.
> 
> 
> Thanks,
> 
> Vishal Suvagia
> 
>



Re: Review Request 73338: RANGER-3275 : Need to update solr-config.xml in the ranger-audits collection config-set

2021-05-06 Thread Vishal Suvagia via Review Board

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

(Updated May 6, 2021, 4:43 p.m.)


Review request for ranger, Ankita Sinha, Dhaval Shah, Dineshkumar Yadav, Gautam 
Borad, Jayendra Parab, Kishor Gollapalliwar, Abhay Kulkarni, Madhan Neethiraj, 
Mehul Parikh, Pradeep Agrawal, Ramesh Mani, Sailaja Polavarapu, and Velmurugan 
Periasamy.


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


Repository: ranger


Description
---

The solrconfig.xml in the config-set for Ranger needs to be updated in correct 
order to use the ttl configuration properly, without which the documents do not 
contain the ttl and will not expire or purge out even after the ttl is set.


Diffs
-

  security-admin/contrib/solr_for_audit_setup/conf/solrconfig.xml 
2216f665fd197e066585fe527645b218ab25a221 


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


Testing
---

Tested the changes locally, ttl is getting applied to the created documents.


Thanks,

Vishal Suvagia