----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31127/#review72781 -----------------------------------------------------------
Ship it! Ship It! - Abraham Elmahrek On Feb. 17, 2015, 7:57 p.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31127/ > ----------------------------------------------------------- > > (Updated Feb. 17, 2015, 7:57 p.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-2109 > https://issues.apache.org/jira/browse/SQOOP-2109 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > There has been several things that I had to fix in order to get this right: > > * Shell packaging was pulling in all dependencies regardless of their scope, > I had to manually switch it from "test" to "runtime" > * Common was directly depending on common-test without properly specifying > testing scope > * Security module was completely missing dependency on com.bueust (that > wasn't problem because 2) has pulled it in transitively) > > > Diffs > ----- > > common/pom.xml 9e593b2 > pom.xml d48389e > security/pom.xml e4e61cc > shell/pom.xml e48c86a > > Diff: https://reviews.apache.org/r/31127/diff/ > > > Testing > ------- > > I've run the packaging target and the shell/lib directory size decreased from > 46MB to 28MB (almost a half!), excluding all the testing dependencies. > > > Thanks, > > Jarek Cecho > >
