sqoop-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Eric Lin <eric....@cloudera.com>
Subject Re: Review Request 57551: SQOOP-2272 - Import decimal columns from mysql to hive 0.14
Date Mon, 20 Mar 2017 08:54:48 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated March 20, 2017, 8:54 a.m.)

Review request for Sqoop, Attila Szabo and Szabolcs Vasas.


change from decimal_column1 to decimal_column in test case in class TestTableDefWriter

Repository: sqoop-trunk


Currently Sqoop converts DECIMAL from RDMS into DOUBLE in Hive, which is not correct as user
will lose precisions. Since Hive supports DECIMAL long ago, we should support DECIMAL to DECIMAL
conversion from Sqoop to Hive.

Diffs (updated)

  src/java/org/apache/sqoop/hive/HiveTypes.java ad00535 
  src/java/org/apache/sqoop/hive/TableDefWriter.java 32fcca3 
  src/test/com/cloudera/sqoop/hive/TestHiveImport.java 1d67a2d 
  src/test/com/cloudera/sqoop/hive/TestTableDefWriter.java 4db629f 
  testdata/hive/scripts/decimalImport.q PRE-CREATION 

Diff: https://reviews.apache.org/r/57551/diff/4/

Changes: https://reviews.apache.org/r/57551/diff/3-4/


Test case + maunaul test


Eric Lin

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