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

[apurtell]

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

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


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

commit 4b2ac4b38e95ee2f863a8d742372a4f3990e2ff9
Author: Andrew Purtell <apurt...@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 <zhang...@apache.org>
    
    Conflicts:
            
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/storefiletracker/StoreFileListFile.java
---
 .../hbase/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);
     }
   }
 }