Github user kumarvishal09 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2487#discussion_r203608964
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/VarLengthColumnPageBase.java
---
@@ -175,24 +182,25 @@ private static ColumnPage
getLVBytesColumnPage(TableSpec.ColumnSpec columnSpec,
throws MemoryException {
// extract length and data, set them to rowOffset and unsafe memory
correspondingly
int rowId = 0;
- List<Integer> rowOffset = new ArrayList<>();
- List<Integer> rowLength = new ArrayList<>();
+ TableSpec.ColumnSpec spec = TableSpec.ColumnSpec
+ .newInstance(columnSpec.getFieldName(), DataTypes.INT,
ColumnType.MEASURE);
+ ColumnPage rowOffset = ColumnPage.newPage(spec, DataTypes.INT, 1024);
--- End diff --
ok
---
