sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Szabolcs Vasas <vasas.szabo...@gmail.com>
Subject Review Request 55558: Fix ordering in column list query of PostgreSQL connector
Date Mon, 16 Jan 2017 09:07:56 GMT

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

Review request for Sqoop and Attila Szabo.

Bugs: SQOOP-3124

Repository: sqoop-trunk


Fix ordering in column list query of PostgreSQL connector


  src/java/org/apache/sqoop/manager/PostgresqlManager.java bd882b9 

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


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

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.


Szabolcs Vasas

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message