This is an automated email from the ASF dual-hosted git repository. amashenkov pushed a commit to branch ignite-26090 in repository https://gitbox.apache.org/repos/asf/ignite-3.git
The following commit(s) were added to refs/heads/ignite-26090 by this push: new 06966070484 minor 06966070484 is described below commit 06966070484c10eedf7147c183af8c6c41a42e09 Author: amashenkov <amashen...@apache.org> AuthorDate: Tue Sep 23 11:46:32 2025 +0300 minor --- .../internal/sql/engine/ItFloatingPointTest.java | 52 +++++++++------------- 1 file changed, 21 insertions(+), 31 deletions(-) diff --git a/modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItFloatingPointTest.java b/modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItFloatingPointTest.java index 064c0d5cb13..d108bceaf3b 100644 --- a/modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItFloatingPointTest.java +++ b/modules/sql-engine/src/integrationTest/java/org/apache/ignite/internal/sql/engine/ItFloatingPointTest.java @@ -17,14 +17,13 @@ package org.apache.ignite.internal.sql.engine; +import static java.util.stream.Collectors.toList; import static org.hamcrest.MatcherAssert.assertThat; import static org.hamcrest.Matchers.contains; -import static org.hamcrest.Matchers.equalTo; +import static org.hamcrest.Matchers.containsInAnyOrder; import static org.junit.jupiter.api.Assertions.assertEquals; import java.util.List; -import java.util.stream.Collectors; -import org.hamcrest.Matchers; import org.junit.jupiter.api.BeforeAll; import org.junit.jupiter.api.Test; import org.junit.jupiter.params.ParameterizedTest; @@ -123,34 +122,14 @@ public class ItFloatingPointTest extends BaseSqlMultiStatementTest { } @Test - void testOrderBySpecialValues() { - { - List<Float> values = sql("SELECT f FROM test ORDER BY f") - .stream().flatMap(List::stream).map(Float.class::cast).collect(Collectors.toList()); - - assertThat(values, Matchers.contains( - equalTo(Float.NEGATIVE_INFINITY), - equalTo(-1.0f), - equalTo(-0.0f), - equalTo(0.0f), - equalTo(1.0f), - equalTo(Float.POSITIVE_INFINITY), - equalTo(Float.NaN) // NaN first or last depending on DB - )); - } - { - List<Double> values = sql("SELECT d FROM test ORDER BY d") - .stream().flatMap(List::stream).map(Double.class::cast).collect(Collectors.toList()); - assertThat(values, Matchers.contains( - equalTo(Double.NEGATIVE_INFINITY), - equalTo(-1.0d), - equalTo(-0.0d), - equalTo(0.0d), - equalTo(1.0d), - equalTo(Double.POSITIVE_INFINITY), - equalTo(Double.NaN) // NaN first or last depending on DB - )); - } + void testOrderBy() { + List<Float> floats = sql("SELECT f FROM test ORDER BY f") + .stream().flatMap(List::stream).map(Float.class::cast).collect(toList()); + List<Double> doubles = sql("SELECT d FROM test ORDER BY d") + .stream().flatMap(List::stream).map(Double.class::cast).collect(toList()); + + assertThat(floats, contains(Float.NEGATIVE_INFINITY, -1.0f, -0.0f, 0.0f, 1.0f, Float.POSITIVE_INFINITY, Float.NaN)); + assertThat(doubles, contains(Double.NEGATIVE_INFINITY, -1.0d, -0.0d, 0.0d, 1.0d, Double.POSITIVE_INFINITY, Double.NaN)); } @Test @@ -187,4 +166,15 @@ public class ItFloatingPointTest extends BaseSqlMultiStatementTest { assertThat(maxRows.get(0), contains(Float.NaN, Double.NaN)); assertThat(avgRows.get(0), contains(Double.NaN, Double.NaN)); } + + @Test + void testGrouping() { + List<Float> floats = sql("SELECT f FROM test GROUP BY f") + .stream().flatMap(List::stream).map(Float.class::cast).collect(toList()); + List<Double> doubles = sql("SELECT d FROM test GROUP BY d") + .stream().flatMap(List::stream).map(Double.class::cast).collect(toList()); + + assertThat(floats, containsInAnyOrder(Float.NEGATIVE_INFINITY, Float.POSITIVE_INFINITY, Float.NaN, -1.0f, 1.0f, 0.0f, -0.0f)); + assertThat(doubles, containsInAnyOrder(Double.NEGATIVE_INFINITY, Double.POSITIVE_INFINITY, Double.NaN, -1.0d, 1.0d, 0.0d, -0.0d)); + } }