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

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


The following commit(s) were added to refs/heads/master by this push:
     new 5f05554d1 [core] Fix warning log when cleaning up a null changelog 
manifest (#3197)
5f05554d1 is described below

commit 5f05554d14cb6af70e2bc36308f8cf1ca6aa32e2
Author: zhourui999 <[email protected]>
AuthorDate: Thu Apr 11 11:49:50 2024 +0800

    [core] Fix warning log when cleaning up a null changelog manifest (#3197)
---
 .../src/main/java/org/apache/paimon/operation/FileDeletionBase.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/paimon-core/src/main/java/org/apache/paimon/operation/FileDeletionBase.java 
b/paimon-core/src/main/java/org/apache/paimon/operation/FileDeletionBase.java
index ded5bedd0..8bd422e53 100644
--- 
a/paimon-core/src/main/java/org/apache/paimon/operation/FileDeletionBase.java
+++ 
b/paimon-core/src/main/java/org/apache/paimon/operation/FileDeletionBase.java
@@ -197,7 +197,7 @@ public abstract class FileDeletionBase {
             Snapshot snapshot, Set<String> skippingSet, boolean 
deleteChangelog) {
         cleanUnusedManifestList(snapshot.baseManifestList(), skippingSet);
         cleanUnusedManifestList(snapshot.deltaManifestList(), skippingSet);
-        if (deleteChangelog) {
+        if (deleteChangelog && snapshot.changelogManifestList() != null) {
             cleanUnusedManifestList(snapshot.changelogManifestList(), 
skippingSet);
         }
         cleanUnusedIndexManifests(snapshot, skippingSet);

Reply via email to