klsince commented on code in PR #10936:
URL: https://github.com/apache/pinot/pull/10936#discussion_r1235967450
##########
pinot-segment-local/src/main/java/org/apache/pinot/segment/local/segment/store/SegmentLocalFSDirectory.java:
##########
@@ -332,11 +332,9 @@ private void prefetchMmapData(PinotDataBuffer buffer) {
PREFETCHED_PAGES.incrementAndGet();
}
} else {
- // pos needs to be long because buffer.size() is 32 bit but
- // adding 4k can make it go over int size
for (long pos = 0; pos < buffer.size() && PREFETCHED_PAGES.get() <
prefetchSlowdownPageLimit;
pos += PAGE_SIZE_BYTES) {
- buffer.getByte((int) pos);
+ buffer.getByte(pos);
Review Comment:
as far as I can see, this is a good catch 👍
--
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: commits-unsubscr...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
