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

    https://github.com/apache/carbondata/pull/1732#discussion_r159461116
  
    --- Diff: 
core/src/main/java/org/apache/carbondata/core/util/DataTypeUtil.java ---
    @@ -775,4 +775,25 @@ public static DataType valueOf(String name) {
         }
       }
     
    -}
    \ No newline at end of file
    +  /**
    +   * Method to type case the data based on modified data type. This method 
will used for
    +   * retrieving the data after change in data type restructure operation
    +   *
    +   * @param data
    +   * @param restructuredDataType
    +   * @param currentDataOffset
    +   * @param length
    +   * @return
    +   */
    +  public static long getDataBasedOnRestructuredDataType(byte[] data, 
DataType restructuredDataType,
    --- End diff --
    
    This case won't exist from short to int or float to double?. Here I can see 
handling only for INT to LONG


---

Reply via email to