Github user maryannxue commented on the issue:

    https://github.com/apache/spark/pull/22732
  
    @srowen What @cloud-fan described is a change introduced in #22259. We can 
fix it by keeping each call to `ScalaReflection.schemaFor` in their own `Try` 
blocks.
    As to `UserDefinedFunction`, after offline discussions with @cloud-fan, we 
decided that there should be no occurrences of calling the constructor without 
setting the `nullableTypes`, so we'll just assert `nullableTypes.length == 
exprs.length`.
    I'll go ahead and fix these two items and update the PR.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to