----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39970/#review105259 -----------------------------------------------------------
Hi Dian, you're indeed correct that the server module depends on all other modules just for the sake of packaging - there are no code dependencies on any of the connectors or repository implementations. The reason we did it this way is that it's easier to maintain few dependencies then copy huge blocks of XML code around to a) copy dependencies to target/ and b) copying them to the distribution tarball. I would prefer not to overly optimize this portion of the packaging code because it's very old and if we want to make major changes, then we should migrate to maven assembly plugin [1] instead (which is the recommended way of doing packaging in maven). Jarcec Links: 1: http://maven.apache.org/plugins/maven-assembly-plugin/ - Jarek Cecho On Nov. 5, 2015, 11:47 a.m., Dian Fu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39970/ > ----------------------------------------------------------- > > (Updated Nov. 5, 2015, 11:47 a.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-2662 > https://issues.apache.org/jira/browse/SQOOP-2662 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > Currently, Sqoop server module has dependencies of all kinds of repositories > and connectors in the pom definition. But from the code level, it doesn't > depends on these modules at all. > > > Diffs > ----- > > connector/connector-ftp/pom.xml 9a70dfc > connector/connector-generic-jdbc/pom.xml 7bdeddb > connector/connector-hdfs/pom.xml a28989c > connector/connector-kafka/pom.xml e0f0684 > connector/connector-kite/pom.xml 41441f5 > connector/connector-sftp/pom.xml 44e64b1 > dist/pom.xml 315a018 > pom.xml ab505f4 > repository/repository-common/pom.xml c740117 > repository/repository-derby/pom.xml a28b036 > repository/repository-mysql/pom.xml 066d76b > repository/repository-postgresql/pom.xml 6550bc2 > server/pom.xml ca068e0 > submission/mapreduce/pom.xml f56437b > test/pom.xml 4e1e197 > > Diff: https://reviews.apache.org/r/39970/diff/ > > > Testing > ------- > > > Thanks, > > Dian Fu > >
