Github user siyuanh commented on a diff in the pull request:

    https://github.com/apache/apex-malhar/pull/323#discussion_r67949068
  
    --- Diff: 
contrib/src/main/java/com/datatorrent/contrib/kafka/AbstractKafkaInputOperator.java
 ---
    @@ -203,6 +204,14 @@ public AbstractKafkaInputOperator()
        */
       protected abstract void emitTuple(Message message);
    --- End diff --
    
    That's why I'm asking can we do that. It does have compatibility issue but 
on the other hand if some one wants to emit the tuple along with partition id 
etc, he/she is forced to implement the emitTuple method as well. Well, we can 
keep it as it is now. @DT-Priyanka can you file a jira to make this change in 
0.9 kafka input operator.  We can change the existing interface for that one    


---
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 [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to