fix

Author: JingsongLi

paimon-core/src/main/java/org/apache/paimon/operation/FileDeletionBase.java

@@ -340,9 +340,11 @@ public Predicate<ExpireFileEntry> createDataFileSkipperForTags(
         if (index >= 0) {
             Snapshot previousTag = taggedSnapshots.get(index);
             if (previousTag.id() != cachedTag) {
-                cachedTag = previousTag.id();
+                cachedTag = 0;
                 cachedTagDataFiles.clear();
                 addMergedDataFiles(cachedTagDataFiles, previousTag);
+                // update cachedTag after read tag successfully
+                cachedTag = previousTag.id();
             }
             return entry -> containsDataFile(cachedTagDataFiles, entry);
         }
@@ -359,7 +361,7 @@ protected List<ManifestFileMeta> tryReadManifestList(String manifestListName) {
         try {
             return manifestList.read(manifestListName);
         } catch (Exception e) {
-            LOG.warn("Failed to read manifest list file " + manifestListName, e);
+            LOG.warn("Failed to read manifest list file {}", manifestListName, e);
             return Collections.emptyList();
         }
     }
@@ -371,7 +373,8 @@ protected List<ManifestFileMeta> tryReadManifestList(String manifestListName) {
     protected void addMergedDataFiles(
             Map<BinaryRow, Map<Integer, Set<String>>> dataFiles, Snapshot snapshot)
             throws IOException {
-        for (ExpireFileEntry entry : readMergedDataFiles(manifestList.readAllManifests(snapshot))) {
+        for (ExpireFileEntry entry :
+                readMergedDataFiles(manifestList.readDataManifests(snapshot))) {
             dataFiles
                     .computeIfAbsent(entry.partition(), p -> new HashMap<>())
                     .computeIfAbsent(entry.bucket(), b -> new HashSet<>())