GJL commented on issue #8583: [FLINK-11820][serialization] SimpleStringSchema 
handle message record which value is null
URL: https://github.com/apache/flink/pull/8583#issuecomment-527374604
 
 
    > Users may not care about the null values in many scenarios
   
   Users may or may not care about _nulls_. If _nulls_ have a meaning for a use 
case then dropping _null_ records can be fatal or lead to unexpected behavior 
in the worst case. According to the [ML 
thread](https://lists.apache.org/thread.html/2991b6b3c520380a9172588bc1f6d7e6d632c3d421458a1b44c71c01@%3Cdev.flink.apache.org%3E)
 Flink currently does not consistently handle _null_ records – changing that 
would be probably a bigger effort. All in all I think it is better to accept 
that `SimpleStringSchema` currently cannot be used if there are _null_ records 
because:
   - There are workarounds, e.g., one does not have to use `SimpleStringSchema`
   - An NPE, which will likely happen while testing the job, is better than 
silent data dropping in production
   
   I will close this issue. Feel free to re-open if you think otherwise.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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

Reply via email to