-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55558/
-----------------------------------------------------------
Review request for Sqoop and Attila Szabo.
Bugs: SQOOP-3124
https://issues.apache.org/jira/browse/SQOOP-3124
Repository: sqoop-trunk
Description
-------
Fix ordering in column list query of PostgreSQL connector
Diffs
-----
src/java/org/apache/sqoop/manager/PostgresqlManager.java bd882b9
Diff: https://reviews.apache.org/r/55558/diff/
Testing
-------
I have run unit tests and PostgreSQL-related third party tests:
ant clean test
ant test -Dthirdparty=true -Dsqoop.thirdparty.lib.dir=libdir -Dmaxmemory=2048m
-DtargetJavaVersion=1.7 -DjavaVersion=1.7
-Dsqoop.test.postgresql.connectstring.host_url=postgresqlserverer
-Dtest.timeout=1000000 -Dtestcase=Postgresql*
The column ordering is correct most of the time without this patch too the bug
is hard to reproduce so it would be very hard to write an automated test to
prove that this patch actually fixed a corner case.
However the currently available test cases can be run successfully which proves
that the patch does not cause a regression.
Thanks,
Szabolcs Vasas