[hbase] branch master 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 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 90d42f2733e HBASE-26979 StoreFileListFile logs frequent stacktraces at 
INFO level (#4373)
90d42f2733e is described below

commit 90d42f2733e9cc6ac627d85aea47bce5ded5c927
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>
---
 .../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);
     }
   }
 }