Skip to content

Commit 8b70c73

Browse files
committed
tests
1 parent 5ec8ec7 commit 8b70c73

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

spark/src/main/scala/org/apache/comet/serde/QueryPlanSerde.scala

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -790,52 +790,52 @@ object QueryPlanSerde extends Logging with CometExprShim {
790790
array.foreach(v => {
791791
val casted = v.asInstanceOf[java.lang.Boolean]
792792
listLiteralBuilder.addBooleanValues(casted)
793-
listLiteralBuilder.addNullMask(casted == null)
793+
listLiteralBuilder.addNullMask(casted != null)
794794
})
795795
case ByteType =>
796796
array.foreach(v => {
797797
val casted = v.asInstanceOf[java.lang.Integer]
798798
listLiteralBuilder.addByteValues(casted)
799-
listLiteralBuilder.addNullMask(casted == null)
799+
listLiteralBuilder.addNullMask(casted != null)
800800
})
801801
case ShortType =>
802802
array.foreach(v => {
803803
val casted = v.asInstanceOf[java.lang.Short]
804804
listLiteralBuilder.addShortValues(
805805
if (casted != null) casted.intValue()
806806
else null.asInstanceOf[java.lang.Integer])
807-
listLiteralBuilder.addNullMask(casted == null)
807+
listLiteralBuilder.addNullMask(casted != null)
808808
})
809809
case IntegerType | DateType =>
810810
array.foreach(v => {
811811
val casted = v.asInstanceOf[java.lang.Integer]
812812
listLiteralBuilder.addIntValues(casted)
813-
listLiteralBuilder.addNullMask(casted == null)
813+
listLiteralBuilder.addNullMask(casted != null)
814814
})
815815
case LongType | TimestampType | TimestampNTZType =>
816816
array.foreach(v => {
817817
val casted = v.asInstanceOf[java.lang.Long]
818818
listLiteralBuilder.addLongValues(casted)
819-
listLiteralBuilder.addNullMask(casted == null)
819+
listLiteralBuilder.addNullMask(casted != null)
820820
})
821821
case FloatType =>
822822
array.foreach(v => {
823823
val casted = v.asInstanceOf[java.lang.Float]
824824
listLiteralBuilder.addFloatValues(casted)
825-
listLiteralBuilder.addNullMask(casted == null)
825+
listLiteralBuilder.addNullMask(casted != null)
826826
})
827827
case DoubleType =>
828828
array.foreach(v => {
829829
val casted = v.asInstanceOf[java.lang.Double]
830830
listLiteralBuilder.addDoubleValues(casted)
831-
listLiteralBuilder.addNullMask(casted == null)
831+
listLiteralBuilder.addNullMask(casted != null)
832832
})
833833
case StringType =>
834834
array.foreach(v => {
835835
val casted = v.asInstanceOf[org.apache.spark.unsafe.types.UTF8String]
836836
listLiteralBuilder.addStringValues(
837837
if (casted != null) casted.toString else "")
838-
listLiteralBuilder.addNullMask(casted == null)
838+
listLiteralBuilder.addNullMask(casted != null)
839839
})
840840
case _: DecimalType =>
841841
array
@@ -846,7 +846,7 @@ object QueryPlanSerde extends Logging with CometExprShim {
846846
com.google.protobuf.ByteString
847847
.copyFrom(casted.toBigDecimal.underlying.unscaledValue.toByteArray)
848848
} else ByteString.EMPTY)
849-
listLiteralBuilder.addNullMask(casted == null)
849+
listLiteralBuilder.addNullMask(casted != null)
850850
})
851851
case _: BinaryType =>
852852
array
@@ -856,7 +856,7 @@ object QueryPlanSerde extends Logging with CometExprShim {
856856
listLiteralBuilder.addBytesValues(if (casted != null) {
857857
com.google.protobuf.ByteString.copyFrom(casted)
858858
} else ByteString.EMPTY)
859-
listLiteralBuilder.addNullMask(casted == null)
859+
listLiteralBuilder.addNullMask(casted != null)
860860
})
861861
}
862862
exprBuilder.setListVal(listLiteralBuilder.build())

0 commit comments

Comments
 (0)