Apache9 commented on a change in pull request #581: HBASE-22888 Use separate classe to deal with streaming read and pread URL: https://github.com/apache/hbase/pull/581#discussion_r323545829
########## File path: hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/StoreFileInfo.java ########## @@ -270,7 +270,10 @@ public StoreFileReader open(FileSystem fs, CacheConfig cacheConf, boolean canUse status = fs.getFileStatus(initialPath); } long length = status.getLen(); - hdfsBlocksDistribution = computeHDFSBlocksDistribution(fs); + if (shared) { Review comment: I know why we need to test this but it is a bit strange... ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services