Github user srdo commented on the issue:

    https://github.com/apache/storm/pull/1999
  
    @hmcl Googled it, it appears that null messages can be sent to delete 
messages in Kafka (see 
https://kafka.apache.org/documentation/#design_compactionbasics). It makes 
sense to drop messages quietly if their values are null. 
    
    It's good to be able to automatically drop them if the RecordTranslator 
returns null as well, I'm mostly worried that bugs in RecordTranslator 
implementations could be hidden by the null drop being enabled by default. It's 
more of a nitpick though.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to