This is an automated email from the ASF dual-hosted git repository. zhangduo pushed a commit to branch branch-2.6 in repository https://gitbox.apache.org/repos/asf/hbase.git
commit 794f90faa1c92d364b651b7a00d28bc8b244d089 Author: guluo <lupeng_n...@qq.com> AuthorDate: Wed May 8 15:31:32 2024 +0800 HBASE-28575 Always printing error log when snapshot table (#5880) Signed-off-by: Duo Zhang <zhang...@apache.org> (cherry picked from commit 2a7aa0d439d8bb17be2cccc057f3d2ec615f7690) --- .../apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java index f746adf0b89..b24f7949404 100644 --- a/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java +++ b/hbase-server/src/main/java/org/apache/hadoop/hbase/master/snapshot/TakeSnapshotHandler.java @@ -252,11 +252,11 @@ public abstract class TakeSnapshotHandler extends EventHandler try { // if the working dir is still present, the snapshot has failed. it is present we delete // it. - if (!workingDirFs.delete(workingDir, true)) { - LOG.error("Couldn't delete snapshot working directory:" + workingDir); + if (workingDirFs.exists(workingDir) && !workingDirFs.delete(workingDir, true)) { + LOG.error("Couldn't delete snapshot working directory: {}", workingDir); } } catch (IOException e) { - LOG.error("Couldn't delete snapshot working directory:" + workingDir); + LOG.error("Couldn't get or delete snapshot working directory: {}", workingDir, e); } if (LOG.isDebugEnabled()) { LOG.debug("Table snapshot journal : \n" + status.prettyPrintJournal());