Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22708#discussion_r225118613
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/JavaTypeInference.scala
 ---
    @@ -271,32 +272,41 @@ object JavaTypeInference {
     
           case c if listType.isAssignableFrom(typeToken) =>
             val et = elementType(typeToken)
    -        MapObjects(
    +        UnresolvedMapObjects(
    --- End diff --
    
    can we exclude other changes except this one? This one is very easy to 
reason about. We did the same thing in `ScalaReflection.`
    
    We need more time to think about the map case, and fix it in 
`ScalaReflection` as well.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to