taklwu commented on PR #4414: URL: https://github.com/apache/hbase/pull/4414#issuecomment-1125519726
> I think you're missing the addition to update HFileReaderImpl, where we created the Runnable to pass to PrefetchExecutor? do you mean `ReaderContextBuilder`? I may be wrong, but configuration was from the `hfs` or `fs` object that should have `hfile.pread.all.bytes.enabled` flag? or do you mean we should have `HFilePreadReader` has a `ReaderContext` that also turn on this addition? https://github.com/apache/hbase/blob/9c8c9e7fbf8005ea89fa9b13d6d063b9f0240443/hbase-server/src/main/java/org/apache/hadoop/hbase/io/hfile/ReaderContextBuilder.java#L92-L95 -- 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]
