----------------------------------------------------------- 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 > >
