LuciferYang commented on pull request #29370: URL: https://github.com/apache/spark/pull/29370#issuecomment-672560495
The main change of Address [fca4fa7](https://github.com/apache/spark/pull/29370/commits/fca4fa788c9f9bef0a465daca0bb4c1b62036b63) as follow: - Add a `LongType` match to `ArraySetLike.nullValueHolder` to resolve problem as follow: ``` java.lang.Integer cannot be cast to java.lang.Long java.lang.ClassCastException: java.lang.Integer cannot be cast to java.lang.Long at scala.runtime.BoxesRunTime.unboxToLong(BoxesRunTime.java:103) at scala.collection.mutable.ArrayBuilder$ofLong.addOne(ArrayBuilder.scala:321) at scala.collection.mutable.Growable.$plus$eq(Growable.scala:38) at scala.collection.mutable.Growable.$plus$eq$(Growable.scala:38) at scala.collection.mutable.ArrayBuilder.$plus$eq(ArrayBuilder.scala:23) at org.apache.spark.sql.catalyst.expressions.GeneratedClass$SpecificMutableProjection.ArrayUnion_0$(Unknown Source) ``` The effect as follow: - Pass `CollectionExpressionsSuite ` **Before** ``` Tests: succeeded 4035, failed 20, canceled 0, ignored 6, pending 0 *** 1 SUITE ABORTED *** *** 20 TESTS FAILED *** ``` **After** ``` Tests: succeeded 4035, failed 17, canceled 0, ignored 6, pending 0 *** 1 SUITE ABORTED *** *** 17 TESTS FAILED *** ``` ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
