> On March 24, 2015, 3:57 p.m., Jarek Cecho wrote: > > test/src/main/java/org/apache/sqoop/test/hive/HiveServerRunner.java, lines > > 130-134 > > <https://reviews.apache.org/r/32429/diff/1/?file=904041#file904041line130> > > > > Seems as excellent use case for our LoggerWriter? > > > > > > https://github.com/apache/sqoop/blob/sqoop2/common-test/src/main/java/org/apache/sqoop/common/test/utils/LoggerWriter.java > > Abraham Elmahrek wrote: > Not sure actually. We would need a LogStream of some kind?
Actualy you're right, those are different APIs. OutputStream and Writer APIs are very close, so we could do something similar, but subsequent JIRA might be better place. - Jarek ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32429/#review77575 ----------------------------------------------------------- On March 24, 2015, 9:01 a.m., Abraham Elmahrek wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32429/ > ----------------------------------------------------------- > > (Updated March 24, 2015, 9:01 a.m.) > > > Review request for Sqoop. > > > Bugs: SQOOP-2016 > https://issues.apache.org/jira/browse/SQOOP-2016 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > commit 1704a3d4ba8bc3b6c6a968c7485374b4c893e453 > Author: Abraham Elmahrek <[email protected]> > Date: Thu Mar 19 12:57:18 2015 -0700 > > SQOOP-2016: Sqoop2: Create integration test for JDBC to Hive > > :000000 100644 0000000... dfe0f43... A > common-test/src/main/java/org/apache/sqoop/common/test/db/HiveProvider.java > :100644 100644 87534c7... 7f0f750... M > common-test/src/main/java/org/apache/sqoop/common/test/utils/NetworkUtils.java > :100644 100644 db8d4e6... 6aa28be... M > connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteDatasetExecutor.java > :100644 100644 44721a8... 0e797f8... M > connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExecutor.java > :100644 100644 e3431bd... c608ca7... M pom.xml > :100644 100644 f743d25... 4af668e... M test/pom.xml > :000000 100644 0000000... 8b355bd... A > test/src/main/java/org/apache/sqoop/test/hive/HiveServerRunner.java > :000000 100644 0000000... fa786e2... A > test/src/main/java/org/apache/sqoop/test/hive/HiveServerRunnerFactory.java > :000000 100644 0000000... 67b39d4... A > test/src/main/java/org/apache/sqoop/test/hive/InternalHiveServerRunner.java > :000000 100644 0000000... 5cc3f8b... A > test/src/main/java/org/apache/sqoop/test/hive/InternalMetastoreServerRunner.java > :000000 100644 0000000... c561c59... A > test/src/main/java/org/apache/sqoop/test/hive/MetastoreServerRunner.java > :000000 100644 0000000... 127ebdf... A > test/src/main/java/org/apache/sqoop/test/hive/MetastoreServerRunnerFactory.java > :100644 100644 648e2f6... 4d27886... M > test/src/main/java/org/apache/sqoop/test/minicluster/TomcatSqoopMiniCluster.java > :000000 100644 0000000... 4388650... A > test/src/main/java/org/apache/sqoop/test/testcases/HiveConnectorTestCase.java > :100644 100644 8ca1c3a... 2698fe4... M > test/src/test/java/org/apache/sqoop/integration/connector/kite/FromRDBMSToKiteTest.java > > > Diffs > ----- > > common-test/src/main/java/org/apache/sqoop/common/test/db/HiveProvider.java > PRE-CREATION > > common-test/src/main/java/org/apache/sqoop/common/test/utils/NetworkUtils.java > 87534c7 > > connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteDatasetExecutor.java > db8d4e6 > > connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExecutor.java > 44721a8 > pom.xml e3431bd > test/pom.xml f743d25 > test/src/main/java/org/apache/sqoop/test/hive/HiveServerRunner.java > PRE-CREATION > test/src/main/java/org/apache/sqoop/test/hive/HiveServerRunnerFactory.java > PRE-CREATION > test/src/main/java/org/apache/sqoop/test/hive/InternalHiveServerRunner.java > PRE-CREATION > > test/src/main/java/org/apache/sqoop/test/hive/InternalMetastoreServerRunner.java > PRE-CREATION > test/src/main/java/org/apache/sqoop/test/hive/MetastoreServerRunner.java > PRE-CREATION > > test/src/main/java/org/apache/sqoop/test/hive/MetastoreServerRunnerFactory.java > PRE-CREATION > > test/src/main/java/org/apache/sqoop/test/minicluster/TomcatSqoopMiniCluster.java > 648e2f6 > > test/src/main/java/org/apache/sqoop/test/testcases/HiveConnectorTestCase.java > PRE-CREATION > > test/src/test/java/org/apache/sqoop/integration/connector/kite/FromRDBMSToKiteTest.java > 8ca1c3a > > Diff: https://reviews.apache.org/r/32429/diff/ > > > Testing > ------- > > mvn clean integration-test > -Dtest=org.apache.sqoop.integration.connector.kite.FromRDBMSToKiteTest#testHiveCities > -DfailIfNoTests=false > > > Thanks, > > Abraham Elmahrek > >
