Github user jackylk commented on a diff in the pull request:
https://github.com/apache/carbondata/pull/2772#discussion_r228049899
--- Diff:
core/src/main/java/org/apache/carbondata/core/datastore/page/UnsafeFixLengthColumnPage.java
---
@@ -534,6 +534,11 @@ private int getEndLoop() {
}
}
+ @Override
+ public long getPageLengthInBytes() {
+ return totalLength;
--- End diff --
For `SafeFixLengthColumnPage`, it can use the function in `ColumnPage`---
