This is an automated email from the ASF dual-hosted git repository.
jshao pushed a commit to branch branch-1.0
in repository https://gitbox.apache.org/repos/asf/gravitino.git
The following commit(s) were added to refs/heads/branch-1.0 by this push:
new df096811b7 [#8380] fix: Fix test in TestPartitionRange (#8423)
df096811b7 is described below
commit df096811b775390230128691483349bc436772c1
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Thu Sep 4 13:54:27 2025 +0800
[#8380] fix: Fix test in TestPartitionRange (#8423)
### What changes were proposed in this pull request?
Fix the assertion to make default order compare to `range2.comparator()`
### Why are the changes needed?
The assertion always true. Make it workable.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
Unit Test
Co-authored-by: MaAng <[email protected]>
Co-authored-by: Jerry Shao <[email protected]>
---
api/src/test/java/org/apache/gravitino/stats/TestPartitionRange.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
a/api/src/test/java/org/apache/gravitino/stats/TestPartitionRange.java
b/api/src/test/java/org/apache/gravitino/stats/TestPartitionRange.java
index e93aaa5ee3..a3ede5d531 100644
--- a/api/src/test/java/org/apache/gravitino/stats/TestPartitionRange.java
+++ b/api/src/test/java/org/apache/gravitino/stats/TestPartitionRange.java
@@ -43,7 +43,7 @@ public class TestPartitionRange {
Assertions.assertFalse(range2.upperPartitionName().isPresent());
Assertions.assertTrue(range2.lowerPartitionName().isPresent());
Assertions.assertEquals("lower", range2.lowerPartitionName().get());
- Assertions.assertEquals(defaultSortOrder, defaultSortOrder);
+ Assertions.assertEquals(defaultSortOrder, range2.comparator());
Assertions.assertEquals(PartitionRange.BoundType.CLOSED,
range2.lowerBoundType().get());
PartitionRange range3 =