-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/29090/#review66545
-----------------------------------------------------------



connector/connector-sdk/src/main/java/org/apache/sqoop/connector/common/AvroDataTypeUtil.java
<https://reviews.apache.org/r/29090/#comment110117>

    this class can be combined into one SqoopAvroUtils, that I am planning to 
address as part of SQOOP-1616



connector/connector-sdk/src/main/java/org/apache/sqoop/connector/common/AvroDataTypeUtil.java
<https://reviews.apache.org/r/29090/#comment110115>

    please call this avroToSqoop or something like it that makes it clear that 
this is converting from avro to sqoop


- Veena Basavaraj


On Dec. 23, 2014, 5:23 p.m., Qian Xu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/29090/
> -----------------------------------------------------------
> 
> (Updated Dec. 23, 2014, 5:23 p.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-1647
>     https://issues.apache.org/jira/browse/SQOOP-1647
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> This JIRA proposes to support reading data from an HDFS dataset. 
> 
> Note that Kite SDK seems have limitation of opening multiple record reader 
> instances by Sqoop 2's demand. This version supports only one 
> partition.https://groups.google.com/a/cloudera.org/forum/#!mydiscussions/cdk-dev/2NJUvh0WmoY
> 
> 
> Diffs
> -----
> 
>   
> connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteConnector.java
>  c864882 
>   
> connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteConnectorError.java
>  d67c8de 
>   
> connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteDatasetExecutor.java
>  9432e4b 
>   
> connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteDatasetPartition.java
>  PRE-CREATION 
>   
> connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteDatasetPartitioner.java
>  PRE-CREATION 
>   
> connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteExtractor.java
>  PRE-CREATION 
>   
> connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteFromDestroyer.java
>  PRE-CREATION 
>   
> connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/KiteFromInitializer.java
>  PRE-CREATION 
>   
> 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/FromJobConfig.java
>  PRE-CREATION 
>   
> connector/connector-kite/src/main/java/org/apache/sqoop/connector/kite/configuration/FromJobConfiguration.java
>  PRE-CREATION 
>   
> connector/connector-kite/src/main/resources/kite-connector-config.properties 
> 27c77b4 
>   
> connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExecutor.java
>  5e4edc5 
>   
> connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteExtractor.java
>  PRE-CREATION 
>   
> connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteFromInitializer.java
>  PRE-CREATION 
>   
> connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteToDestroyer.java
>  4051fda 
>   
> connector/connector-kite/src/test/java/org/apache/sqoop/connector/kite/TestKiteToInitializer.java
>  5f0525d 
>   connector/connector-sdk/pom.xml 38c217a 
>   
> connector/connector-sdk/src/main/java/org/apache/sqoop/connector/common/AvroDataTypeUtil.java
>  PRE-CREATION 
>   pom.xml afdeb75 
> 
> Diff: https://reviews.apache.org/r/29090/diff/
> 
> 
> Testing
> -------
> 
> Added new test cases.
> 
> 
> Thanks,
> 
> Qian Xu
> 
>

Reply via email to