Github user gvramana commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2487#discussion_r202321022
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/VarLengthColumnPageBase.java
---
@@ -201,31 +209,32 @@ private static ColumnPage
getComplexLVBytesColumnPage(TableSpec.ColumnSpec colum
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 --
This default can be equal NUMBER_OF_ROWS_PER_BLOCKLET_COLUMN_PAGE_DEFAULT
---