Github user Fokko commented on the issue:

    https://github.com/apache/spark/pull/21596
  
    I will first fix the tests, and then assert the performance if you agree. 
Changing the annotations might impact the performance.
    
    Regarding the difference between `NON_NULL`, `NON_ABSENT`, `NON_EMPTY`, 
please refer to Jackson:  
https://fasterxml.github.io/jackson-annotations/javadoc/2.7/com/fasterxml/jackson/annotation/JsonInclude.Include.html
    The `NON_EMPTY` makes the most sense to me. I agree that we should keep the 
behaviour like it is right now.
    
    `NON_EMPTY`:  Value that indicates that only properties with null value, or 
what is considered empty, are not to be included. 


---

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

Reply via email to