Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2978#discussion_r240524219
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/chunk/store/impl/safe/AbstractNonDictionaryVectorFiller.java
---
@@ -83,20 +84,37 @@ public StringVectorFiller(int numberOfRows, int
actualDataLength) {
@Override
public void fillVector(byte[] data, CarbonColumnVector vector) {
// start position will be used to store the current data position
+ boolean invertedIndex = vector instanceof
ColumnarVectorWrapperDirectWithInvertedIndex
+ || vector instanceof SequentialFill;
--- End diff --
It seems SequentialFill is not subclass of CarbonColumnVector?
---
