-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/26276/
-----------------------------------------------------------
(Updated Nov. 10, 2014, 3:09 p.m.)
Review request for Sqoop.
Changes
-------
this rev renames size to length ( again as said before a nitpick to be
consistent with java string class)
Bugs: SQOOP-1703
https://issues.apache.org/jira/browse/SQOOP-1703
Repository: sqoop-SQOOP-1367
Description
-------
ssee jira.
rename type to column type ( since it is enum representing columns)
rename size on stirng to length, again a nitpick, since it is in line with java
string class
No functional change
Diffs (updated)
-----
common/src/main/java/org/apache/sqoop/json/util/SchemaSerialization.java
4b4b932
common/src/main/java/org/apache/sqoop/schema/Schema.java 3aa3aea
common/src/main/java/org/apache/sqoop/schema/type/AbstractString.java d84c7ec
common/src/main/java/org/apache/sqoop/schema/type/Array.java cbf8f6e
common/src/main/java/org/apache/sqoop/schema/type/Binary.java 53e50b8
common/src/main/java/org/apache/sqoop/schema/type/Bit.java 3c522bf
common/src/main/java/org/apache/sqoop/schema/type/Column.java 97bd303
common/src/main/java/org/apache/sqoop/schema/type/ColumnType.java
PRE-CREATION
common/src/main/java/org/apache/sqoop/schema/type/Date.java 3d1bffe
common/src/main/java/org/apache/sqoop/schema/type/DateTime.java 502f328
common/src/main/java/org/apache/sqoop/schema/type/Decimal.java ffd3eab
common/src/main/java/org/apache/sqoop/schema/type/Enum.java 48fd048
common/src/main/java/org/apache/sqoop/schema/type/FixedPoint.java 77adfb1
common/src/main/java/org/apache/sqoop/schema/type/FloatingPoint.java ceff011
common/src/main/java/org/apache/sqoop/schema/type/Map.java 2dd70d8
common/src/main/java/org/apache/sqoop/schema/type/Set.java c9a658f
common/src/main/java/org/apache/sqoop/schema/type/Text.java 550e2b6
common/src/main/java/org/apache/sqoop/schema/type/Time.java 095fac0
common/src/main/java/org/apache/sqoop/schema/type/Type.java 7fca3c8
common/src/main/java/org/apache/sqoop/schema/type/Unsupported.java e09bc48
connector/connector-sdk/src/main/java/org/apache/sqoop/connector/idf/CSVIntermediateDataFormat.java
e65edd9
Diff: https://reviews.apache.org/r/26276/diff/
Testing
-------
Thanks,
Veena Basavaraj