Github user manishgupta88 commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2772#discussion_r227304248
--- 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 --
Instead of sending the encodedBytes send the length of encodedBytes as we
are not making any other use of encodedBytes
---
