anoopsjohn commented on a change in pull request #1881:
URL: https://github.com/apache/hbase/pull/1881#discussion_r438022749



##########
File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java
##########
@@ -622,7 +622,8 @@ void setDataBlockEncoderInTest(HFileDataBlockEncoder 
blockEncoder) {
       for (HStoreFile storeFile : results) {
         if (compactedStoreFiles.contains(storeFile.getPath().getName())) {
           LOG.warn("Clearing the compacted storefile {} from {}", storeFile, 
this);
-          storeFile.getReader().close(true);
+          storeFile.getReader().close(storeFile.getCacheConf() != null ?

Review comment:
       While closing the region, doing the evict caused delay in closure. But 
here are we seeing some impact because of this evict? This will happen in async 
way from the Discharger thread right? Just trying to understand.




----------------------------------------------------------------
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:
[email protected]


Reply via email to