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

Review request for Sqoop and Bilung Lee.

Summary
-------

Fix for SQOOP-383

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

Diffs

/src/java/org/apache/sqoop/tool/VersionTool.java 1197384
/src/scripts/write-version-info.sh 1197384

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

Testing
-------

Compiled and executed version tool.

Thanks,

Arvind

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

Ship it!

Look good. Thanks, Arvind!

• Bilung

On 2011-11-04 01:57:17, Arvind Prabhakar wrote:

-----------------------------------------------------------

This is an automatically generated e-mail. To reply, visit:

https://reviews.apache.org/r/2720/

-----------------------------------------------------------

(Updated 2011-11-04 01:57:17)

Review request for Sqoop and Bilung Lee.

Summary

-------

Fix for SQOOP-383

This addresses bug SQOOP-383.

https://issues.apache.org/jira/browse/SQOOP-383

Diffs

-----

/src/java/org/apache/sqoop/tool/VersionTool.java 1197384

/src/scripts/write-version-info.sh 1197384

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

Testing

-------

Compiled and executed version tool.

Thanks,

Arvind

Integrated in Sqoop-jdk-1.6 #58 (See https://builds.apache.org/job/Sqoop-jdk-1.6/58/)
SQOOP-383 Version tool is not working

blee : http://svn.apache.org/viewvc/?view=rev&rev=1197390
Files :

• /incubator/sqoop/trunk/CHANGES.txt
• /incubator/sqoop/trunk/src/java/org/apache/sqoop/tool/VersionTool.java
• /incubator/sqoop/trunk/src/scripts/write-version-info.sh

Patch committed. Thanks, Arvind!