[ https://issues.apache.org/jira/browse/SQOOP-3396?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16707471#comment-16707471 ]
ASF GitHub Bot commented on SQOOP-3396: --------------------------------------- Github user szvasas commented on a diff in the pull request: https://github.com/apache/sqoop/pull/60#discussion_r237907664 --- Diff: src/java/org/apache/sqoop/hive/HiveTypes.java --- @@ -79,8 +85,42 @@ public static String toHiveType(int sqlType) { default: // TODO(aaron): Support BINARY, VARBINARY, LONGVARBINARY, DISTINCT, // BLOB, ARRAY, STRUCT, REF, JAVA_OBJECT. - return null; + return null; + } + } + + private static String mapDecimalsToHiveType(int sqlType, SqoopOptions options) { + if (options.getConf().getBoolean(ConfigurationConstants.PROP_ENABLE_PARQUET_LOGICAL_TYPE_DECIMAL, false) + && (sqlType == Types.NUMERIC || sqlType == Types.DECIMAL)){ --- End diff -- Do we need to test for NUMERIC and DECIMAL types again here? > Add parquet numeric support for Parquet in Hive import > ------------------------------------------------------ > > Key: SQOOP-3396 > URL: https://issues.apache.org/jira/browse/SQOOP-3396 > Project: Sqoop > Issue Type: Sub-task > Reporter: Fero Szabo > Assignee: Fero Szabo > Priority: Major > -- This message was sent by Atlassian JIRA (v7.6.3#76005)