-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29565/
-----------------------------------------------------------
(Updated Jan. 5, 2015, 10:25 a.m.)
Review request for Sqoop.
Changes
-------
1. Added several test cases for configuration objects
2. Fixed a NPE when dataset uri is null
3. Fixed a bug that file format cannot be null
4. Fixed a validation issue of hostPortString
Bugs: SQOOPO-1751
https://issues.apache.org/jira/browse/SQOOPO-1751
Repository: sqoop-sqoop2
Description
-------
This is a follow up jira of SQOOP-1588.
Two things should be changed:
1. The File Output Format should be moved to ToJobConfig.
2. Credential configs should be kept in LinkConfig (such as HDFS host and port,
which should be read from environment or here, not from toJobConfig.uri)
Note that the current upgrade api does not allow `toJobConfig` or
`fromJobConfig` to access original `linkConfig` (also in reverse direction), so
it is not able to read valueFoo from linkConfig into toJobConfig. I fallback to
a default value manually.
Diffs (updated)
-----
connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteConnectorUpgrader.java
d3b9f95
connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteLoader.java
0a46f4a
connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteToDestroyer.java
25912b4
connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteToInitializer.java
11233a8
connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/configuration/LinkConfig.java
89bd9b3
connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/configuration/ToJobConfig.java
70b7dc3
connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/util/InputValidation.java
53fab02
connector/connector-kite/src/main/resources/kite-connector-config.properties
27c77b4
connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteConnectorConfigurations.java
PRE-CREATION
connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteConnectorUpgrader.java
PRE-CREATION
connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteToDestroyer.java
4051fda
Diff: https://reviews.apache.org/r/29565/diff/
Testing
-------
Added test cases to check config objects are expected after upgrade.
Thanks,
Qian Xu
