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

    https://github.com/apache/carbondata/pull/3001#discussion_r245915288
  
    --- Diff: 
integration/presto/src/main/java/org/apache/carbondata/presto/CarbondataPageSourceProvider.java
 ---
    @@ -79,13 +80,31 @@
       @Override
       public ConnectorPageSource createPageSource(ConnectorTransactionHandle 
transactionHandle,
           ConnectorSession session, ConnectorSplit split, List<ColumnHandle> 
columns) {
    -    this.queryId = ((CarbondataSplit)split).getQueryId();
    +    CarbondataSplit carbondataSplit = (CarbondataSplit) split;
    --- End diff --
    
    already reverted the modification. but it is difficult to reuse 
PrestoCarbonVectorizedRecordReader for streaming file. 


---

Reply via email to