wgtmac commented on code in PR #1177:
URL: https://github.com/apache/parquet-mr/pull/1177#discussion_r1417500605
##########
parquet-column/src/main/java/org/apache/parquet/internal/column/columnindex/OffsetIndexBuilder.java:
##########
@@ -70,6 +72,14 @@ public long getFirstRowIndex(int pageIndex) {
public int getPageOrdinal(int pageIndex) {
return pageIndex;
}
+
+ @Override
+ public Optional<Long> getUnencodedByteArrayDataBytes(int pageIndex) {
+ if (unencodedByteArrayDataBytes == null ||
unencodedByteArrayDataBytes.length == 0) {
Review Comment:
Here the length means number of pages. If there is no page, then there is no
unencodedByteArrayDataBytes either.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
