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)