Github user xubo245 commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2334#discussion_r190103982
  
    --- Diff: 
integration/presto/src/main/java/org/apache/carbondata/presto/PrestoFilterUtil.java
 ---
    @@ -92,14 +69,15 @@ private static DataType 
Spi2CarbondataTypeMapper(CarbondataColumnHandle carbonda
         else if (colType == DateType.DATE) return DataTypes.DATE;
         else if (colType == TimestampType.TIMESTAMP) return 
DataTypes.TIMESTAMP;
         else if 
(colType.equals(DecimalType.createDecimalType(carbondataColumnHandle.getPrecision(),
    -        carbondataColumnHandle.getScale()))) return DataTypes
    -        .createDecimalType(carbondataColumnHandle.getPrecision(),
    -            carbondataColumnHandle.getScale());
    +            carbondataColumnHandle.getScale()))) return DataTypes
    --- End diff --
    
    Why change this one?


---

Reply via email to