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

    https://github.com/apache/carbondata/pull/2546#discussion_r204654611
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/datastore/page/UnsafeFixLengthColumnPage.java
 ---
    @@ -495,6 +495,8 @@ private int getEndLoop() {
           return totalLength / ByteUtil.SIZEOF_BYTE;
         } else if (dataType == DataTypes.SHORT) {
           return totalLength / ByteUtil.SIZEOF_SHORT;
    +    } else if (dataType == DataTypes.SHORT_INT) {
    --- End diff --
    
    I think this is also required for SafeFixLengthColumnPage please check 


---

Reply via email to