From fd88fa77565b13e8cc9c842a9cebb217453d6b93 Mon Sep 17 00:00:00 2001 From: DieterDP <90392398+DieterDP-ng@users.noreply.github.com> Date: Fri, 1 Mar 2024 13:06:19 +0100 Subject: [PATCH] HBASE-28408 Rephrase confusing log message (#5718) Assume a user has a series of backups: Full1, Inc2, Inc3, where a table has not changed between Full1 and Inc3, but has changed after Inc3. When restoring that table to Inc3, a log warning was outputted mentioning there was no need for a restore. This message in fact means there is no need for the incremental restore portion of the restore process. Signed-off-by: Nihal Jain (cherry picked from commit 102c1b684a893dd3475eb2c047414cbe395f9b8e) --- .../apache/hadoop/hbase/backup/impl/RestoreTablesClient.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/RestoreTablesClient.java b/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/RestoreTablesClient.java index 05685c8e091e..654fe343e27d 100644 --- a/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/RestoreTablesClient.java +++ b/hbase-backup/src/main/java/org/apache/hadoop/hbase/backup/impl/RestoreTablesClient.java @@ -173,7 +173,8 @@ private void restoreImages(BackupImage[] images, TableName sTable, TableName tTa } if (dirList.isEmpty()) { - LOG.warn("Nothing has changed, so there is no need to restore '" + sTable + "'"); + LOG.info("No incremental changes since full backup for '" + sTable + + "', skipping incremental restore step."); return; }