-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29025/
-----------------------------------------------------------
Review request for Sqoop.
Bugs: SQOOP-1882
https://issues.apache.org/jira/browse/SQOOP-1882
Repository: sqoop-sqoop2
Description
-------
see JIRA
It fixes few things
JobManager currently ignores the TO connector IDF.
IDF cannot assume the type to be String
Using Set for jars than list, so we can avoid duplicates
Diffs
-----
connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcFromInitializer.java
adf03d9
connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcToInitializer.java
930cfec
connector/connector-kafka/src/main/java/org/apache/sqoop/connector/kafka/KafkaToInitializer.java
e1b065a
connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteToInitializer.java
40b3abf
connector/connector-sdk/src/main/java/org/apache/sqoop/connector/idf/CSVIntermediateDataFormat.java
dbe193d
connector/connector-sdk/src/main/java/org/apache/sqoop/connector/idf/IntermediateDataFormat.java
93698a8
connector/connector-sdk/src/test/java/org/apache/sqoop/connector/idf/TestCSVIntermediateDataFormat.java
83a95ec
core/src/main/java/org/apache/sqoop/driver/JobManager.java 01073d4
core/src/main/java/org/apache/sqoop/driver/JobRequest.java 8c1cc95
core/src/test/java/org/apache/sqoop/driver/TestJobRequest.java 9eff8a4
execution/mapreduce/src/main/java/org/apache/sqoop/execution/mapreduce/MapreduceExecutionEngine.java
9b3eb44
execution/mapreduce/src/main/java/org/apache/sqoop/job/MRJobConstants.java
67021a8
execution/mapreduce/src/main/java/org/apache/sqoop/job/io/SqoopWritable.java
336ab97
execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopMapper.java
7434243
execution/mapreduce/src/main/java/org/apache/sqoop/job/mr/SqoopOutputFormatLoadExecutor.java
d664337
execution/mapreduce/src/test/java/org/apache/sqoop/job/TestMapReduce.java
256c34d
execution/mapreduce/src/test/java/org/apache/sqoop/job/TestMatching.java
67c8525
execution/mapreduce/src/test/java/org/apache/sqoop/job/mr/TestSqoopOutputFormatLoadExecutor.java
d897125
spi/src/main/java/org/apache/sqoop/job/etl/Initializer.java 4dd6d5b
Diff: https://reviews.apache.org/r/29025/diff/
Testing
-------
yes
Thanks,
Veena Basavaraj
