You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by ravipesala <gi...@git.apache.org> on 2018/01/03 16:05:52 UTC

[GitHub] carbondata pull request #1732: [CARBONDATA-1946] Exception thrown after alte...

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


---