[GitHub] carbondata pull request #2297: [CARBONDATA-2471]Added support for No Diction...

[ravipesala]

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

    https://github.com/apache/carbondata/pull/2297#discussion_r187551301
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/util/DataTypeUtil.java ---
    @@ -449,6 +457,16 @@ public static Object 
getDataBasedOnDataTypeForNoDictionaryColumn(byte[] dataInBy
               return null;
             }
             return ByteUtil.toLong(dataInBytes, 0, dataInBytes.length) * 1000L;
    +      } else if (actualDataType == DataTypes.DOUBLE) {
    +        if (isEmptyByteArray(dataInBytes)) {
    +          return null;
    +        }
    +        return Double.longBitsToDouble(ByteUtil.toLong(dataInBytes, 0, 
dataInBytes.length));
    --- End diff --
    
    Move to ByteUtil


---