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 b614edbe6b [core] Fix validation messages for comparing retained 
snapshots & changelog (#5777)
b614edbe6b is described below

commit b614edbe6bbf0a0e91f228da42ff986295ca416f
Author: Yubin Li <[email protected]>
AuthorDate: Fri Jun 20 10:15:20 2025 +0800

    [core] Fix validation messages for comparing retained snapshots & changelog 
(#5777)
---
 .../src/main/java/org/apache/paimon/table/ExpireChangelogImpl.java   | 5 ++++-
 .../src/main/java/org/apache/paimon/table/ExpireSnapshotsImpl.java   | 5 ++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git 
a/paimon-core/src/main/java/org/apache/paimon/table/ExpireChangelogImpl.java 
b/paimon-core/src/main/java/org/apache/paimon/table/ExpireChangelogImpl.java
index cf11a2ba14..1b42f2339b 100644
--- a/paimon-core/src/main/java/org/apache/paimon/table/ExpireChangelogImpl.java
+++ b/paimon-core/src/main/java/org/apache/paimon/table/ExpireChangelogImpl.java
@@ -105,7 +105,10 @@ public class ExpireChangelogImpl implements 
ExpireSnapshots {
         }
 
         Preconditions.checkArgument(
-                retainMax >= retainMin, "retainMax must greater than 
retainMin.");
+                retainMax >= retainMin,
+                String.format(
+                        "retainMax (%s) must not be less than retainMin (%s).",
+                        retainMax, retainMin));
 
         // the min snapshot to retain from 'changelog.num-retained.max'
         // (the maximum number of snapshots to retain)
diff --git 
a/paimon-core/src/main/java/org/apache/paimon/table/ExpireSnapshotsImpl.java 
b/paimon-core/src/main/java/org/apache/paimon/table/ExpireSnapshotsImpl.java
index 5560b7a82e..01c7e74966 100644
--- a/paimon-core/src/main/java/org/apache/paimon/table/ExpireSnapshotsImpl.java
+++ b/paimon-core/src/main/java/org/apache/paimon/table/ExpireSnapshotsImpl.java
@@ -102,7 +102,10 @@ public class ExpireSnapshotsImpl implements 
ExpireSnapshots {
         }
 
         Preconditions.checkArgument(
-                retainMax >= retainMin, "retainMax must greater than 
retainMin.");
+                retainMax >= retainMin,
+                String.format(
+                        "retainMax (%s) must not be less than retainMin (%s).",
+                        retainMax, retainMin));
 
         // the min snapshot to retain from 'snapshot.num-retained.max'
         // (the maximum number of snapshots to retain)

Reply via email to