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 =

Reply via email to