[ 
https://issues.apache.org/jira/browse/CARBONDATA-32?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15360038#comment-15360038
 ] 

ASF GitHub Bot commented on CARBONDATA-32:
------------------------------------------

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

    https://github.com/apache/incubator-carbondata/pull/12#discussion_r69374118
  
    --- Diff: 
core/src/main/java/org/carbondata/scan/result/preparator/impl/RawQueryResultPreparatorImpl.java
 ---
    @@ -92,7 +91,7 @@ public 
RawQueryResultPreparatorImpl(QueryExecutorProperties executerProperties,
               int noDictionaryColumnIndex = 0;
               for (int i = 0; i < dimSize; i++) {
                 if 
(!queryDimensions[i].getDimension().hasEncoding(Encoding.DICTIONARY)) {
    -              row[order[i]] = DataTypeUtil.getDataBasedOnDataType(
    +              row[order[i]] = 
org.carbondata.core.util.DataTypeUtil.getDataBasedOnDataType(
    --- End diff --
    
    import the package


> Refactor DataType related code
> ------------------------------
>
>                 Key: CARBONDATA-32
>                 URL: https://issues.apache.org/jira/browse/CARBONDATA-32
>             Project: CarbonData
>          Issue Type: Bug
>            Reporter: Jacky Li
>            Priority: Minor
>
> 1. Merge two DataType
> 2. Merge tow DataTypeUtil
> 3. Remove DataSetUtil



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to