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-500719263
 
 
   @lamber-ken 
   
   The contract of `deserialize()` says:
   ```
            *
         * @return The deserialized message as an object (null if the message 
cannot be deserialized).
         */
   ```
   
   That is, returning `null` from the deserialization schema denotes an 
erroneous condition. Null records in Kafka can have valid semantics. However, 
if we apply your patch, null records will be discarded by the Kafka connector 
(i.e., they won't be consumed by downstream operators). Let me know if you see 
a solution to this problem.

----------------------------------------------------------------
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