[hbase] branch branch-2 updated: HBASE-26979 StoreFileListFile logs frequent stacktraces at INFO level (#4373)

[ap...@apache.org]

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

apurtell pushed a commit to branch branch-2
in repository https://gitbox.apache.org/repos/asf/hbase.git


The following commit(s) were added to refs/heads/branch-2 by this push:
     new 6813a5c502e HBASE-26979 StoreFileListFile logs frequent stacktraces at INFO level (#4373)
6813a5c502e is described below

commit 6813a5c502efc6d8165c3b6242363e80d266f184
Author: Andrew Purtell <ap...@apache.org>
AuthorDate: Wed Apr 27 18:02:00 2022 -0700

    HBASE-26979 StoreFileListFile logs frequent stacktraces at INFO level (#4373)
    
    Signed-off-by: Duo Zhang <zh...@apache.org>
---
 .../hbase/regionserver/storefiletracker/StoreFileListFile.java      | 6 +++---
 1 file changed, 3 insertions(+), 3 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 5ed35c7beae..9976ba10ea3 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
@@ -200,8 +200,8 @@ class StoreFileListFile {
           lists[i] = load(files.get(i));
           loaded = true;
         } catch (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);
         }
       }
       if (loaded) {
@@ -258,7 +258,7 @@ class StoreFileListFile {
     } 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
-      LOG.debug("failed to delete old track file {}, not a big deal, just ignore", e);
+      LOG.debug("Failed to delete old track file {}, ignoring the exception", e);
     }
   }
 }