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

Review request for Sqoop and Jarek Cecho.


Summary
-------

The maven build is broken due to the missing SqoopVersion.java file(s) that are 
generated via the ant build. Also, on my development machine running Ubuntu 
11.04 I am seeing a failure in the maven compiler plugin caused by the 
cross-referencing of namespace for DBOutputFormat class and its subclass.

This change introduces the maven-antrun-plugin to generate the 
SqoopVersion.java files, uses the org.codehaus.mojo:build-helper-maven-plugin 
to add the generated source directory to the compiler source path and also 
fixes the DBOutputFormat to avoid the maven compiler crash problem.


This addresses bug SQOOP-392.
    https://issues.apache.org/jira/browse/SQOOP-392


Diffs
-----

  /pom.xml 1205214 
  /src/java/org/apache/sqoop/mapreduce/db/DBOutputFormat.java 1205214 

Diff: https://reviews.apache.org/r/2920/diff


Testing
-------

The maven build invokes rat plugin which fails due to missing or incorrect 
license headers. This can be locally avoided by passing in the property such as 
-Drat.numUnapprovedLicenses=1000. This issue should be addressed by a different 
JIRA.

Also tested the build via ant - and all tests passed successfully.


Thanks,

Arvind

Reply via email to