----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27004/ -----------------------------------------------------------
(Updated Oct. 22, 2014, 1:18 p.m.) Review request for Sqoop. Changes ------- integration tests pass as well.! added some java docs to explain the configurations used in the 5 steps of the job lifecyle. Repository: sqoop-sqoop2 Description ------- see JIRA for details. also it fixes a lot of warnings in the code base. Diffs (updated) ----- connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsConnector.java e63e464 connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsDestroyer.java 74b1cb8 connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsExtractor.java 2c8b6c8 connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsFromDestroyer.java PRE-CREATION connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsFromInitializer.java PRE-CREATION connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsInitializer.java bb5e353 connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsLoader.java 660418d connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsPartitioner.java f40459f connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsToDestroyer.java PRE-CREATION connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/HdfsToInitializer.java PRE-CREATION connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/configuration/LinkConfig.java 5d48a29 connector/connector-hdfs/src/main/java/org/apache/sqoop/connector/hdfs/configuration/LinkConfiguration.java c0cd336 connector/connector-hdfs/src/main/resources/hdfs-connector-config.properties 9b8c6ba connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestExtractor.java c6d2f90 connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestLoader.java 552a751 connector/connector-hdfs/src/test/java/org/apache/sqoop/connector/hdfs/TestPartitioner.java 9d177ec connector/connector-sdk/src/main/java/org/apache/sqoop/connector/common/EmptyConfiguration.java PRE-CREATION core/src/main/java/org/apache/sqoop/driver/JobManager.java 51e562c execution/mapreduce/src/test/java/org/apache/sqoop/job/TestMapReduce.java 6d0dcb4 execution/mapreduce/src/test/java/org/apache/sqoop/job/TestMatching.java 665a65b spi/src/main/java/org/apache/sqoop/job/etl/Destroyer.java a133106 spi/src/main/java/org/apache/sqoop/job/etl/Extractor.java d6c186d spi/src/main/java/org/apache/sqoop/job/etl/Initializer.java 5c48fc3 spi/src/main/java/org/apache/sqoop/job/etl/Loader.java cc32ada spi/src/main/java/org/apache/sqoop/job/etl/Partitioner.java 57507df Diff: https://reviews.apache.org/r/27004/diff/ Testing ------- yes tests pass. Thanks, Veena Basavaraj
