Re: Review Request 73938: RANGER-3696 add log4j2 binding to elasticsearch plugin

[bhavik patel <bh...@gmail.com>]

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



Please rebase the patch.

- bhavik patel


On May 10, 2022, 2:23 a.m., Kirby Zhou wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73938/
> -----------------------------------------------------------
> 
> (Updated May 10, 2022, 2:23 a.m.)
> 
> 
> Review request for ranger, Dhaval Shah, Dineshkumar Yadav, Kishor Gollapalliwar, Abhay Kulkarni, Madhan Neethiraj, Mateen Mansoori, Mehul Parikh, Pradeep Agrawal, Ramesh Mani, and Velmurugan Periasamy.
> 
> 
> Bugs: RANGER-3696
>     https://issues.apache.org/jira/browse/RANGER-3696
> 
> 
> Repository: ranger
> 
> 
> Description
> -------
> 
> elasticsearch plugin use slf4j, but elasticsearch takes log4j2.
> And there is no log4j and slf4j adapter existing.
> 
> So need to add slf4j-api and its log4j2 binding to dist package
> 
> 
> Diffs
> -----
> 
>   distro/src/main/assembly/plugin-elasticsearch.xml 0b8aaee27b8330cf1b61a03021318ef4e26891ee 
>   pom.xml 7eac176c79799983101e3c2676568fc11e1d0799 
>   ranger-elasticsearch-plugin-shim/pom.xml 5a02be9b5d71c887985a216e31e30494287b7f84 
>   ranger-plugin-classloader/pom.xml bcc02812adbb765f9f2c3c62cd0bc35084e041fb 
> 
> 
> Diff: https://reviews.apache.org/r/73938/diff/3/
> 
> 
> Testing
> -------
> 
> Need somebody to verify, I can just do very simple test.
> 
> 
> Thanks,
> 
> Kirby Zhou
> 
>