You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@iceberg.apache.org by fo...@apache.org on 2024/03/12 13:34:00 UTC

(iceberg) 01/01: Fix `TestStrictMetricsEvaluator` assertion message

This is an automated email from the ASF dual-hosted git repository.

fokko pushed a commit to branch fd-patch-test
in repository https://gitbox.apache.org/repos/asf/iceberg.git

commit edd14db8605bcca9b73c6fccfbea62bd3d6859ef
Author: Fokko Driesprong <fo...@apache.org>
AuthorDate: Tue Mar 12 14:33:53 2024 +0100

    Fix `TestStrictMetricsEvaluator` assertion message
    
    All values (`5` and `6`) are not between the upper and lower bound of `[30, 79]`.
---
 .../java/org/apache/iceberg/expressions/TestStrictMetricsEvaluator.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/api/src/test/java/org/apache/iceberg/expressions/TestStrictMetricsEvaluator.java b/api/src/test/java/org/apache/iceberg/expressions/TestStrictMetricsEvaluator.java
index 2266389aba..82aaff0214 100644
--- a/api/src/test/java/org/apache/iceberg/expressions/TestStrictMetricsEvaluator.java
+++ b/api/src/test/java/org/apache/iceberg/expressions/TestStrictMetricsEvaluator.java
@@ -592,7 +592,7 @@ public class TestStrictMetricsEvaluator {
     boolean shouldRead =
         new StrictMetricsEvaluator(SCHEMA, notIn("id", INT_MIN_VALUE - 25, INT_MIN_VALUE - 24))
             .eval(FILE);
-    assertThat(shouldRead).as("Should not match: all values !=5 and !=6").isTrue();
+    assertThat(shouldRead).as("Should match: all values !=5 and !=6").isTrue();
 
     shouldRead =
         new StrictMetricsEvaluator(SCHEMA, notIn("id", INT_MIN_VALUE - 1, INT_MIN_VALUE))