Sanil15 commented on a change in pull request #984: Samza 2116: Making 
sendTo(table), sendTo(stream) non-terminal
URL: https://github.com/apache/samza/pull/984#discussion_r276774703
 
 

 ##########
 File path: 
samza-api/src/main/java/org/apache/samza/operators/MessageStream.java
 ##########
 @@ -276,13 +282,19 @@
 
   /**
    * Sends messages in this {@link MessageStream} to a {@link Table}. The type 
of input message is expected
-   * to be {@link KV}, otherwise a {@link ClassCastException} will be thrown.
-   *
+   * to be {@link KV}, otherwise a {@link ClassCastException} will be thrown. 
After doing a put to table this operator
+   * propagates the message in this {@link MessageStream} to the next chained 
operator.
+   * <p>
+   *  Note: It is a guarantee that message is first written to table and then 
only it is propagated to next operator
 
 Review comment:
   What do you mean by "It is a guarantee that message is first written to 
table". Are you saying that we flush the table before calling next? 
   No, just put()
   
   Will do the necessary updates

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