[iceberg] branch master updated: Core: Remove confusing log from RemoveSnapshots (#5478)

[blue]

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

blue pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/iceberg.git


The following commit(s) were added to refs/heads/master by this push:
     new c8831580dd Core: Remove confusing log from RemoveSnapshots (#5478)
c8831580dd is described below

commit c8831580ddd7641303afa62b7c82f461c6ae4ddc
Author: Ajantha Bhat <ajanthab...@gmail.com>
AuthorDate: Sat Oct 15 02:18:14 2022 +0530

    Core: Remove confusing log from RemoveSnapshots (#5478)
---
 core/src/main/java/org/apache/iceberg/RemoveSnapshots.java | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/core/src/main/java/org/apache/iceberg/RemoveSnapshots.java 
b/core/src/main/java/org/apache/iceberg/RemoveSnapshots.java
index 68c5a2b4e3..fa6fcdf414 100644
--- a/core/src/main/java/org/apache/iceberg/RemoveSnapshots.java
+++ b/core/src/main/java/org/apache/iceberg/RemoveSnapshots.java
@@ -310,8 +310,6 @@ class RemoveSnapshots implements ExpireSnapshots {
 
     if (cleanExpiredFiles) {
       cleanExpiredSnapshots();
-    } else {
-      LOG.info("Cleaning up manifest and data files disabled, leaving them in 
place");
     }
   }
 
@@ -327,6 +325,9 @@ class RemoveSnapshots implements ExpireSnapshots {
       incrementalCleanup = current.refs().size() == 1;
     }
 
+    LOG.info(
+        "Cleaning up expired files (local, {})", incrementalCleanup ? 
"incremental" : "reachable");
+
     FileCleanupStrategy cleanupStrategy =
         incrementalCleanup
             ? new IncrementalFileCleanup(