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
---