This is an automated email from the ASF dual-hosted git repository.
lakshsingla pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/druid.git
The following commit(s) were added to refs/heads/master by this push:
new f5c5573da89 Revert "Fix FloatFirstVectorAggregationTest"
f5c5573da89 is described below
commit f5c5573da89a924875e209dc14b5ac7019e0822f
Author: Laksh Singla <[email protected]>
AuthorDate: Tue Mar 19 14:39:09 2024 +0530
Revert "Fix FloatFirstVectorAggregationTest"
This reverts commit c40bb2c8f071043a87ede54d9dc56d024774ccfd.
---
.../query/aggregation/first/FloatFirstVectorAggregationTest.java | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git
a/processing/src/test/java/org/apache/druid/query/aggregation/first/FloatFirstVectorAggregationTest.java
b/processing/src/test/java/org/apache/druid/query/aggregation/first/FloatFirstVectorAggregationTest.java
index a95e89bd32a..28044de5e4e 100644
---
a/processing/src/test/java/org/apache/druid/query/aggregation/first/FloatFirstVectorAggregationTest.java
+++
b/processing/src/test/java/org/apache/druid/query/aggregation/first/FloatFirstVectorAggregationTest.java
@@ -64,7 +64,9 @@ public class FloatFirstVectorAggregationTest extends
InitializedNullHandlingTest
new SerializablePairLongFloat(2345300L, 4.2F)
};
- private final SerializablePairLongFloat[] nullPairs = {null, null, null,
null};
+ private final SerializablePairLongFloat[] null_pairs = {
+ null, null, null, null
+ };
@@ -128,7 +130,7 @@ public class FloatFirstVectorAggregationTest extends
InitializedNullHandlingTest
@Override
public Object[] getObjectVector()
{
- return nullPairs;
+ return null_pairs;
}
@Override
@@ -285,7 +287,7 @@ public class FloatFirstVectorAggregationTest extends
InitializedNullHandlingTest
target1.aggregate(buf, 0, 0, VALUES.length);
Pair<Long, Float> result = (Pair<Long, Float>) target1.get(buf, 0);
Assert.assertEquals(Long.MAX_VALUE, result.lhs.longValue());
- Assert.assertEquals(NullHandling.defaultFloatValue(), result.rhs);
+ Assert.assertNull(result.rhs);
}
@Test
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
