Github user szvasas commented on a diff in the pull request: https://github.com/apache/sqoop/pull/60#discussion_r237795529 --- Diff: src/java/org/apache/sqoop/hive/HiveTypes.java --- @@ -37,16 +42,28 @@ private static final String HIVE_TYPE_STRING = "STRING"; private static final String HIVE_TYPE_BOOLEAN = "BOOLEAN"; private static final String HIVE_TYPE_BINARY = "BINARY"; + private static final String HIVE_TYPE_DECIMAL = "DECIMAL"; public static final Log LOG = LogFactory.getLog(HiveTypes.class.getName()); private HiveTypes() { } + + public static String toHiveType(int sqlType, SqoopOptions options) { + + if (options.getConf().getBoolean(ConfigurationConstants.PROP_ENABLE_PARQUET_LOGICAL_TYPE_DECIMAL, false) --- End diff -- I think this check could go to the original toHiveType to the NUMERIC/DECIMAL switch branch in a separate method.
---