Ngone51 commented on issue #23651: [SPARK-26730][SQL] Strip redundant 
AssertNotNull for ExpressionEncoder's serializer
URL: https://github.com/apache/spark/pull/23651#issuecomment-459397164
 
 
   > Why? 
   
   We'll still add `AssertNotNull` for the input(`BoundReference`) when we do 
null safe transform for it:
   
   ```
   val nullSafeSerializer = objSerializer.transformUp {
     case r: BoundReference =>
     // For input object of Product type, we can't encode it to row if it's 
null, as Spark SQL
     // doesn't allow top-level row to be null, only its columns can be null.
    AssertNotNull(r, Seq("top level Product or row object"))
   }
   ```
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

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

Reply via email to