You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by vj...@apache.org on 2020/05/29 13:34:40 UTC
[hbase] branch branch-2.2 updated: HBASE-24470 : Add store file
info when log meessages in HStore. (#1804)
This is an automated email from the ASF dual-hosted git repository.
vjasani pushed a commit to branch branch-2.2
in repository https://gitbox.apache.org/repos/asf/hbase.git
The following commit(s) were added to refs/heads/branch-2.2 by this push:
new 604d7bd HBASE-24470 : Add store file info when log meessages in HStore. (#1804)
604d7bd is described below
commit 604d7bdfb82834f63f2f06ff30702bd6a59f7b0b
Author: xincunSong <36...@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 <bi...@gmail.com>
Signed-off-by: Viraj Jasani <vj...@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 ec7c0a4..bb17b43 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
@@ -615,7 +615,7 @@ public class HStore implements Store, HeapSize, StoreConfigInformation, Propagat
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;
@@ -2754,13 +2754,15 @@ public class HStore implements Store, HeapSize, StoreConfigInformation, Propagat
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;