[hbase] branch master updated: HBASE-24470 : Add store file info when log meessages in HStore. (#1804)

[vjasani]

This is an automated email from the ASF dual-hosted git repository.

vjasani pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/master by this push:
     new da2e03b  HBASE-24470 : Add store file info when log meessages in 
HStore. (#1804)
da2e03b is described below

commit da2e03bb3b3dbd0511e236b5390d4f2680cf3071
Author: xincunSong <365724...@qq.com>
AuthorDate: Fri May 29 21:28:57 2020 +0800

    HBASE-24470 : Add store file info when log meessages in HStore. (#1804)
    
    Signed-off-by: binlijin <binli...@gmail.com>
    Signed-off-by: Viraj Jasani <vjas...@apache.org>
---
 .../main/java/org/apache/hadoop/hbase/regionserver/HStore.java    | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java
index 65602eb..1f926a2 100644
--- 
a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java
+++ 
b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HStore.java
@@ -608,7 +608,7 @@ public class HStore implements Store, HeapSize, 
StoreConfigInformation,
             file.closeStoreFile(evictOnClose);
           }
         } catch (IOException e) {
-          LOG.warn("Could not close store file", e);
+          LOG.warn("Could not close store file {}", file, e);
         }
       }
       throw ioe;
@@ -2815,13 +2815,15 @@ public class HStore implements Store, HeapSize, 
StoreConfigInformation,
       file.initReader();
       length = file.getReader().length();
     } catch (IOException e) {
-      LOG.trace("Failed to open reader when trying to compute store file size, 
ignoring", e);
+      LOG.trace("Failed to open reader when trying to compute store file size 
for {}, ignoring",
+        file, e);
     } finally {
       try {
         file.closeStoreFile(
             file.getCacheConf() != null ? 
file.getCacheConf().shouldEvictOnClose() : true);
       } catch (IOException e) {
-        LOG.trace("Failed to close reader after computing store file size, 
ignoring", e);
+        LOG.trace("Failed to close reader after computing store file size for 
{}, ignoring",
+          file, e);
       }
     }
     return length;