ajothomas commented on a change in pull request #1560:
URL: https://github.com/apache/samza/pull/1560#discussion_r787163691
##########
File path: samza-api/src/main/java/org/apache/samza/operators/MessageStream.java
##########
@@ -291,12 +291,34 @@
* scheme when propogated to next operator.
*
* @param table the table to write messages to
- * @param args additional arguments passed to the table
* @param <K> the type of key in the table
* @param <V> the type of record value in the table
* @return this {@link MessageStream}
*/
- <K, V> MessageStream<KV<K, V>> sendTo(Table<KV<K, V>> table, Object ...
args);
+ <K, V> MessageStream<KV<K, V>> sendTo(Table<KV<K, V>> table);
+
+ /**
+ * Allows sending update messages in this {@link MessageStream} to a {@link
Table} and then propagates this
+ * {@link MessageStream} to the next chained operator. The type of input
message is expected to be {@link KV},
+ * otherwise a {@link ClassCastException} will be thrown. The value is an
UpdateMessage- update and default value.
+ * Defaults are optional and can be used if the Remote Table integration
supports inserting a default through PUT in
+ * the event an update fails due to an existing record being absent.
+ * The user also needs to pass UpdateOptions argument which defines whether
the update is an update only operation
Review comment:
Done
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
