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

    https://github.com/apache/carbondata/pull/2687#discussion_r216123298
  
    --- Diff: 
store/sdk/src/main/java/org/apache/carbondata/sdk/file/AvroCarbonWriter.java ---
    @@ -714,6 +758,11 @@ private static DataType 
getMappingDataTypeForCollectionRecord(Schema childSchema
               int precision = ((LogicalTypes.Decimal) 
childSchema.getLogicalType()).getPrecision();
               int scale = ((LogicalTypes.Decimal) 
childSchema.getLogicalType()).getScale();
               return DataTypes.createDecimalType(precision, scale);
    +        } else if (logicalType == null && 
childSchema.getObjectProp("logicalType")
    --- End diff --
    
    Consider the previous comment for decimal here also.


---

Reply via email to