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

Reply via email to