Github user sv71294 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2412#discussion_r199068497
--- Diff:
integration/presto/src/main/java/org/apache/carbondata/presto/CarbondataPageSourceProvider.java
---
@@ -103,7 +102,7 @@ private PrestoCarbonVectorizedRecordReader
createReader(ConnectorSplit split,
CarbonIterator iterator = queryExecutor.execute(queryModel);
readSupport.initialize(queryModel.getProjectionColumns(),
queryModel.getTable());
PrestoCarbonVectorizedRecordReader reader = new
PrestoCarbonVectorizedRecordReader(queryExecutor, queryModel,
- (AbstractDetailQueryResultIterator) iterator);
+ (AbstractDetailQueryResultIterator) iterator, readSupport);
--- End diff --
added dictionary read support for streamReaders
---