From 4b2ac4b38e95ee2f863a8d742372a4f3990e2ff9 Mon Sep 17 00:00:00 2001 From: Andrew Purtell Date: Wed, 27 Apr 2022 18:02:00 -0700 Subject: [PATCH] HBASE-26979 StoreFileListFile logs frequent stacktraces at INFO level (#4373) Signed-off-by: Duo Zhang Conflicts: hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/storefiletracker/StoreFileListFile.java --- .../regionserver/storefiletracker/StoreFileListFile.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/storefiletracker/StoreFileListFile.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/storefiletracker/StoreFileListFile.java index ced01187b69..42233e4aa7a 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/storefiletracker/StoreFileListFile.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/storefiletracker/StoreFileListFile.java @@ -120,8 +120,8 @@ class StoreFileListFile { try { lists[i] = load(trackFiles[i]); } catch (FileNotFoundException | EOFException e) { - // this is normal case, so use info and do not log stacktrace - LOG.info("Failed to load track file {}: {}", trackFiles[i], e.toString()); + // this is normal case, so just log at debug + LOG.debug("EOF loading track file {}, ignoring the exception", trackFiles[i], e); } } int winnerIndex = select(lists); @@ -162,8 +162,8 @@ class StoreFileListFile { fs.delete(trackFiles[nextTrackFile], false); } catch (IOException e) { // we will create new file with overwrite = true, so not a big deal here, only for speed up - // loading as we do not need to read this file when loading(we will hit FileNotFoundException) - LOG.debug("failed to delete old track file {}, not a big deal, just ignore", e); + // loading as we do not need to read this file when loading + LOG.debug("Failed to delete old track file {}, ignoring the exception", e); } } }