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

Review request for Sqoop, Attila Szabo and Szabolcs Vasas.


Repository: sqoop-trunk


Description
-------

The following command:

sqoop import --connect jdbc:mysql://localhost/test --username root --password 
'cloudera' --table decimal_table  -m 1 --driver com.mysql.jdbc.Driver --verbose 
--hive-import --hive-database default --hive-table decimal_table 
--hive-overwrite --map-column-hive a='DECIMAL(10,4)'

will fail with below error:

{code}
17/03/13 18:42:09 DEBUG sqoop.Sqoop: Malformed mapping.  Column mapping should 
be the form key=value[,key=value]*
java.lang.IllegalArgumentException: Malformed mapping.  Column mapping should 
be the form key=value[,key=value]*
        at 
org.apache.sqoop.SqoopOptions.parseColumnMapping(SqoopOptions.java:1333)
        at 
org.apache.sqoop.SqoopOptions.setMapColumnHive(SqoopOptions.java:1349)
        at 
org.apache.sqoop.tool.BaseSqoopTool.applyHiveOptions(BaseSqoopTool.java:1198)
        at org.apache.sqoop.tool.ImportTool.applyOptions(ImportTool.java:1011)
        at org.apache.sqoop.tool.SqoopTool.parseArguments(SqoopTool.java:435)
        at org.apache.sqoop.Sqoop.run(Sqoop.java:135)
        at org.apache.hadoop.util.ToolRunner.run(ToolRunner.java:70)
        at org.apache.sqoop.Sqoop.runSqoop(Sqoop.java:183)
        at org.apache.sqoop.Sqoop.runTool(Sqoop.java:234)
        at org.apache.sqoop.Sqoop.runTool(Sqoop.java:243)
        at org.apache.sqoop.Sqoop.main(Sqoop.java:252)
Malformed mapping.  Column mapping should be the form key=value[,key=value]*
{code}

--map-column-hive should support DECIMAL(10,5) format.


Diffs
-----

  src/java/org/apache/sqoop/SqoopOptions.java b33b54b 
  src/test/com/cloudera/sqoop/hive/TestTableDefWriter.java 4db629f 


Diff: https://reviews.apache.org/r/57576/diff/1/


Testing
-------

Added test case + manual testing


Thanks,

Eric Lin

Reply via email to