Github user hvanhovell commented on a diff in the pull request: https://github.com/apache/spark/pull/20757#discussion_r173150013 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala --- @@ -121,6 +121,19 @@ object ScalaReflection extends ScalaReflection { case _ => false } + def classForNativeTypeOf(dt: DataType): Class[_] = dt match { --- End diff -- Shouldn't this match `CodeGenerator.boxedType` in terms of functionality? See my previous comments, but I am also missing complex types (struct, map & array).
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org