Apache
/
druid
mirror of https://github.com/apache/druid.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Fix incorrect tests in Sting first/last serde's null handling (#15657) 

Fixes a couple of incorrect test cases, that got merged accidentally
This commit is contained in:
Laksh Singla 2024-01-10 19:16:12 +05:30 committed by GitHub
parent d623756c66
commit 0b91cc4db2
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
processing/src/test/java/org/apache/druid/query/aggregation/BackwardCompatibleSerializablePairLongStringSimpleStagedSerdeTest.java
View File

@ -59,7 +59,7 @@ public class BackwardCompatibleSerializablePairLongStringSimpleStagedSerdeTest
SerializablePairLongString value = new SerializablePairLongString(Long.MAX_VALUE, null);
// Write using the older serde, read using the newer serde
Assert.assertEquals(
new SerializablePairLongString(Long.MAX_VALUE, ""),
new SerializablePairLongString(Long.MAX_VALUE, null),
readUsingSerde(writeUsingSerde(value, OLDER_SERDE), SERDE)
);
// Write using the newer serde, read using the older serde
@ -77,7 +77,7 @@ public class BackwardCompatibleSerializablePairLongStringSimpleStagedSerdeTest
SerializablePairLongString value = new SerializablePairLongString(Long.MAX_VALUE, "");
// Write using the older serde, read using the newer serde
Assert.assertEquals(
new SerializablePairLongString(Long.MAX_VALUE, ""),
new SerializablePairLongString(Long.MAX_VALUE, null),
readUsingSerde(writeUsingSerde(value, OLDER_SERDE), SERDE)
);
// Write using the newer serde, read using the older serde