-----------------------------------------------------------
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

Reply via email to