----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/59844/ -----------------------------------------------------------
(Updated June 29, 2017, 5:08 p.m.) Review request for Sqoop. Changes ------- Fixing DirectPostgreSQLExportManualTest Bugs: SQOOP-3197 https://issues.apache.org/jira/browse/SQOOP-3197 Repository: sqoop-trunk Description ------- PostgreSQL third party tests look like they support configurable database parameters (e.g. they load the database user, name and password from a system property) but the property names in the test cases do not match the property names in the build.xml and because of that these parameters cannot be specified from the command line. Diffs (updated) ----- build.xml 2903b7d146e3c4f18bc46f52a2945b59fa789f4c src/test/com/cloudera/sqoop/manager/DirectPostgreSQLExportManualTest.java 52095ef296449c0c2bcce2653c5e797c222b5e20 src/test/com/cloudera/sqoop/manager/PostgresqlExportTest.java 38d6ba69da062907a89536727bd9b4c4d0cd1903 src/test/com/cloudera/sqoop/manager/PostgresqlExternalTableImportTest.java 0515e098a870635628f4ea34cc4340292eab9744 src/test/com/cloudera/sqoop/manager/PostgresqlImportTest.java 5872d259475d017074d67ffac266fdb48e59c4b7 Diff: https://reviews.apache.org/r/59844/diff/2/ Changes: https://reviews.apache.org/r/59844/diff/1-2/ Testing ------- Ran unit and third party test cases. Thanks, Szabolcs Vasas