> On Jan. 5, 2021, 5:33 p.m., Velmurugan Periasamy wrote: > > pom.xml > > Line 116 (original), 116 (patched) > > <https://reviews.apache.org/r/72941/diff/2/?file=2243888#file2243888line116> > > > > Can you please make sure this is not packaged? I see > > spotbugs-annotations-3.1.9.jar is getting packaged. > > bhavik patel wrote: > This jar is getting packaged without this patch and after analyzing it is > found that it is pulled by "zookeeper" dependency. > > Velmurugan Periasamy wrote: > Thanks for the analysis. My suggestion is to exclude > spotbugs-annotations-3.1.9.jar - either in is this patch or via a separate > jira.
Will create separate jira to exclude spotbugs-annotations-3.1.9.jar file. - bhavik ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72941/#review222406 ----------------------------------------------------------- On Jan. 5, 2021, 6:05 a.m., bhavik patel wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72941/ > ----------------------------------------------------------- > > (Updated Jan. 5, 2021, 6:05 a.m.) > > > Review request for ranger, Ankita Sinha, Dhaval Shah, Dineshkumar Yadav, > Gautam Borad, Abhay Kulkarni, Madhan Neethiraj, Mehul Parikh, Pradeep > Agrawal, Ramesh Mani, Sailaja Polavarapu, and Velmurugan Periasamy. > > > Bugs: RANGER-3030 > https://issues.apache.org/jira/browse/RANGER-3030 > > > Repository: ranger > > > Description > ------- > > Since Findbugs is no longer maintained, so it will be good to use the latest > version of the Spotbugs Maven plugin as a part of good coding practice. > > > Diffs > ----- > > dev-support/findbugsIncludeFile.xml 8623906bd > dev-support/spotbugsIncludeFile.xml PRE-CREATION > pom.xml 11a695065 > > > Diff: https://reviews.apache.org/r/72941/diff/2/ > > > Testing > ------- > > erified "mvn spotbugs:check" successful. > > > Thanks, > > bhavik patel > >