[GitHub] carbondata pull request #1307: [CARBONDATA-1433] Added Vectorized Reader for...

[jackylk]

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

    https://github.com/apache/carbondata/pull/1307#discussion_r137283427
  
    --- Diff: 
integration/presto/src/main/java/org/apache/carbondata/presto/CarbonTypeUtil.java
 ---
    @@ -0,0 +1,34 @@
    +package org.apache.carbondata.presto;
    +
    +import org.apache.carbondata.core.metadata.datatype.DataType;
    +
    +import org.apache.spark.sql.types.DataTypes;
    --- End diff --
    
    should not depend on spark in presto integration module


---