Github user ravipesala commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2897#discussion_r243810091
--- Diff:
hadoop/src/main/java/org/apache/carbondata/hadoop/util/CarbonVectorizedRecordReader.java
---
@@ -68,6 +67,10 @@
private AbstractDetailQueryResultIterator iterator;
private QueryModel queryModel;
+ //This holds mapping of fetch index with respect to project col index.
+ // it is used when same col is used in projection many times.So need to
fetch only that col.
+ private List<Integer> projectionMapping = new ArrayList<>();
--- End diff --
Better use an array to have fast lookups instead of list
---