----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39847/#review104825 -----------------------------------------------------------
Ship it! ranger-plugin-classloader/src/main/java/org/apache/ranger/plugin/classloader/RangerPluginClassLoader.java (line 61) <https://reviews.apache.org/r/39847/#comment163078> Please check if this catch and throw is needed? - Madhan Neethiraj On Nov. 2, 2015, 7:36 a.m., Ramesh Mani wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39847/ > ----------------------------------------------------------- > > (Updated Nov. 2, 2015, 7:36 a.m.) > > > Review request for ranger and Madhan Neethiraj. > > > Repository: ranger > > > Description > ------- > > RANGER-715:Fix issues reported by coverity test in Ranger Pluging ClassLoader > > > Diffs > ----- > > > ranger-plugin-classloader/src/main/java/org/apache/ranger/plugin/classloader/RangerPluginClassLoader.java > eafcd27 > > Diff: https://reviews.apache.org/r/39847/diff/ > > > Testing > ------- > > Tested in Local VM > > > Thanks, > > Ramesh Mani > >
