QiangCai commented on a change in pull request #3691: [CARBONDATA-3760]Enable 
prefetch in presto to improve query performance
URL: https://github.com/apache/carbondata/pull/3691#discussion_r402176992
 
 

 ##########
 File path: 
integration/presto/src/main/prestosql/org/apache/carbondata/presto/CarbondataPageSource.java
 ##########
 @@ -356,7 +356,6 @@ private PrestoCarbonVectorizedRecordReader 
createReaderForColumnar(HiveSplit car
     QueryModel queryModel = createQueryModel(carbonSplit, tableHandle, 
columns, conf);
     if (isDirectVectorFill) {
       queryModel.setDirectVectorFill(true);
-      queryModel.setPreFetchData(false);
 
 Review comment:
   No, we can enable it.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to