sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mengwei Ding" <mengwei.d...@cloudera.com>
Subject Re: Review Request 12092: SQOOP-1073 Sqoop2: Introduce schema for transferred data
Date Mon, 01 Jul 2013 16:36:24 GMT

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



common/src/main/java/org/apache/sqoop/schema/Schema.java
<https://reviews.apache.org/r/12092/#comment46307>

    Hi, Jarce, I find a tiny error here. I think the StringUtils in the org.apache.commons.utils
package rather than the lang package.


- Mengwei Ding


On June 29, 2013, 9:11 p.m., Jarek Cecho wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/12092/
> -----------------------------------------------------------
> 
> (Updated June 29, 2013, 9:11 p.m.)
> 
> 
> Review request for Sqoop.
> 
> 
> Bugs: SQOOP-1073
>     https://issues.apache.org/jira/browse/SQOOP-1073
> 
> 
> Repository: sqoop-sqoop2
> 
> 
> Description
> -------
> 
> Full implementation of the first schema drop. It is adding all facilities required to
describe, create and transfer the schema. Further tweaking will be required and will be done
in follow up JIRAs.
> 
> 
> Diffs
> -----
> 
>   client/src/main/java/org/apache/sqoop/client/core/Constants.java 06f8162e76122451bf0e52f8749af610a37837e8

>   client/src/main/java/org/apache/sqoop/client/utils/SubmissionDisplayer.java 360a9318f268ba7b266307df2f9f1de2d6fd4ade

>   client/src/main/resources/client-resource.properties 8a570a1621ce37225d2e72b70aa332564a3ffa0a

>   common/pom.xml 2921800aa1cde5f8df476ca14ca3218de11884c5 
>   common/src/main/java/org/apache/sqoop/json/SchemaBean.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/json/SubmissionBean.java 91f2007f00b5c0e6cefa8a7feb03257b3b13866c

>   common/src/main/java/org/apache/sqoop/json/util/SchemaSerialization.java PRE-CREATION

>   common/src/main/java/org/apache/sqoop/model/MSubmission.java e15c46651aad7bfe3969d051390756a14dd338b3

>   common/src/main/java/org/apache/sqoop/schema/Schema.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/SchemaError.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/AbstractComplexType.java PRE-CREATION

>   common/src/main/java/org/apache/sqoop/schema/type/AbstractDateTime.java PRE-CREATION

>   common/src/main/java/org/apache/sqoop/schema/type/AbstractNumber.java PRE-CREATION

>   common/src/main/java/org/apache/sqoop/schema/type/AbstractString.java PRE-CREATION

>   common/src/main/java/org/apache/sqoop/schema/type/Array.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Binary.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Bit.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Column.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Date.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/DateTime.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Decimal.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Enum.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/FixedPoint.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/FloatingPoint.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Map.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Set.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Text.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Time.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Type.java PRE-CREATION 
>   common/src/main/java/org/apache/sqoop/schema/type/Unsupported.java PRE-CREATION 
>   common/src/test/java/org/apache/sqoop/json/TestSchemaBean.java PRE-CREATION 
>   common/src/test/java/org/apache/sqoop/json/TestSubmissionBean.java 19a5bbad7ef4714deb118f69c14e24cf36ccf390

>   common/src/test/java/org/apache/sqoop/json/util/TestSchemaSerialization.java PRE-CREATION

>   connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcConnectorError.java
f2ac979d1fe128de385025534b7979789685ed2a 
>   connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcExportInitializer.java
40a777499492a3b18f1fee1fd96222c77b8868ce 
>   connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/GenericJdbcImportInitializer.java
3e9789c35e1e56d53d30257801cc8ce6194bbc0c 
>   connector/connector-generic-jdbc/src/main/java/org/apache/sqoop/connector/jdbc/util/SqlTypesUtils.java
PRE-CREATION 
>   connector/connector-generic-jdbc/src/test/java/org/apache/sqoop/connector/jdbc/TestImportInitializer.java
9f4269a0ef7579b72a437e167a15a91408e5537d 
>   core/src/main/java/org/apache/sqoop/framework/JobManager.java 5a2f490c6702edd359e7867d30df76bd745a1824

>   spi/src/main/java/org/apache/sqoop/job/etl/Initializer.java 346b84c34d5ed71faf74650d842c7581f5e880e5

> 
> Diff: https://reviews.apache.org/r/12092/diff/
> 
> 
> Testing
> -------
> 
> Both unit and integration tests seems to be passing.
> 
> 
> Thanks,
> 
> Jarek Cecho
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message