mynameborat commented on a change in pull request #1560:
URL: https://github.com/apache/samza/pull/1560#discussion_r788896284
##########
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);
Review comment:
so what is the option for them to get the same functionality from the
new API? i.e., UpdateOptions supports these potential params that can be passed
as var args is it?
--
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]
