Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2772#discussion_r228050566
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/encoding/ColumnPageEncoder.java
---
@@ -100,17 +100,17 @@ private DataChunk2 buildPageMetadata(ColumnPage
inputPage, byte[] encodedBytes)
throws IOException {
DataChunk2 dataChunk = new DataChunk2();
dataChunk.setData_page_length(encodedBytes.length);
- fillBasicFields(inputPage, dataChunk);
+ fillBasicFields(inputPage, dataChunk, encodedBytes);
--- End diff --
ok, now I remove this parameter and use dataChunk.getData_page_length
instead
---
