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

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 9e593b2e23d54db34ad5737c08637365792e6537 
  pom.xml d48389ee60c1fbc5fa76e85a577d48f1e2032aec 
  security/pom.xml e4e61ccaa43aa62c7db66fe72ade3bc25b4155aa 
  shell/pom.xml e48c86a20f0bb5bf0ba8a5f12d563bb4c6f0f26a 

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

Reply via email to