----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12397/#review22946 -----------------------------------------------------------
Ship it! +1 - Good catch - Venkat Ranganathan On July 10, 2013, 2:35 a.m., Jarek Cecho wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12397/ > ----------------------------------------------------------- > > (Updated July 10, 2013, 2:35 a.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-1129 > https://issues.apache.org/jira/browse/SQOOP-1129 > > > Repository: sqoop-trunk > > > Description > ------- > > The test code was expecting different file name than is being created. The > changed file name was introduced in SQOOP-1078. > > > Diffs > ----- > > src/test/com/cloudera/sqoop/manager/PostgresqlImportTest.java > 7414199b548dc4b5f6520c156d8cd6714c18681c > > Diff: https://reviews.apache.org/r/12397/diff/ > > > Testing > ------- > > Testcase PostgresqlImportTest is now working correctly. I've used variant of > following command to verify that: > > ant clean test -Dtestcase=PostgresqlImportTest > -Dsqoop.test.postgresql.connectstring.host_url=jdbc:postgresql://postgresql/ > > > Thanks, > > Jarek Cecho > >
