-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54030/
-----------------------------------------------------------
(Updated Nov. 29, 2016, 11:22 a.m.)
Review request for Sqoop and Attila Szabo.
Changes
-------
Removed <p> from Licence header
Bugs: SQOOP-3055
https://issues.apache.org/jira/browse/SQOOP-3055
Repository: sqoop-trunk
Description
-------
SQOOP-3055 MySQL tests are ignoring specified username, password and dbname,
trying to connect to specified db using currentuser (using
System.getEnv("USER") or whoami). You should now be able to specify the user
(and will have to when running Mysql tests), password, hostname and database
name for the MSQL tests (and they won't ignore it and use current users
anymore).
Diffs (updated)
-----
build.xml 6e8d393
src/test/com/cloudera/sqoop/manager/DirectMySQLExportTest.java 79e90e2
src/test/com/cloudera/sqoop/manager/DirectMySQLTest.java d0f0c50
src/test/com/cloudera/sqoop/manager/JdbcMySQLExportTest.java 86b9aab
src/test/com/cloudera/sqoop/manager/MySQLAllTablesTest.java bad3c29
src/test/com/cloudera/sqoop/manager/MySQLCompatTest.java 675b6b8
src/test/com/cloudera/sqoop/manager/MySQLFreeFormQueryTest.java 935d948
src/test/com/cloudera/sqoop/manager/MySQLLobAvroImportTest.java df78142
src/test/com/cloudera/sqoop/manager/MySQLTestUtils.java 0af79d6
src/test/org/apache/sqoop/manager/mysql/MySqlCallExportTest.java 6bf939b
src/test/org/apache/sqoop/manager/mysql/MySqlColumnEscapeImportTest.java
87cd389
Diff: https://reviews.apache.org/r/54030/diff/
Testing
-------
Ran all MySQL tests with hadoopversion 260 and hbaseprofile 95: ant test
-Dhadoopversion=260 -Dhbaseprofile=95 -Dthirdparty=true
-Dsqoop.thirdparty.lib.dir=3rdpartylibdir
-Dsqoop.test.mysql.connectstring.host_url=mysqlhost
-Dsqoop.test.mysql.databasename=sqoop_repo -Dsqoop.test.mysql.password=sqoop
-Dsqoop.test.mysql.username=sqoop
Thanks,
Anna Szonyi