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

    https://github.com/apache/carbondata/pull/1795#discussion_r161792809
  
    --- Diff: 
processing/src/main/java/org/apache/carbondata/processing/sort/sortdata/SortDataRows.java
 ---
    @@ -270,7 +270,7 @@ private void writeDataToFile(Object[][] 
recordHolderList, int entryCountLocal, F
                 } else if (dataType == DataTypes.SHORT) {
                   stream.writeShort((Short) value);
                 } else if (dataType == DataTypes.INT) {
    -              stream.writeInt((Integer) value);
    +              stream.writeInt(Integer.parseInt(value.toString()));
    --- End diff --
    
    But what is the root cause of this bug? I think your change (convert to 
string and then int) is like hiding this bug.


---

Reply via email to