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



##########
File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java
##########
@@ -2741,7 +2741,8 @@ private void removeCompactedfiles(Collection<HStoreFile> 
compactedfiles)
             LOG.trace("Closing and archiving the file {}", file);
             // Copy the file size before closing the reader
             final long length = r.length();
-            r.close(true);
+            r.close(file.getCacheConf() != null ?

Review comment:
       Thank you for the clarification. I understood that. Will change the 
patch according to your review. Thanks.




----------------------------------------------------------------
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