ableegoldman commented on a change in pull request #10573:
URL: https://github.com/apache/kafka/pull/10573#discussion_r619557786
##########
File path: streams/src/main/java/org/apache/kafka/streams/StreamsConfig.java
##########
@@ -291,23 +293,35 @@
* <p>
* Enabling exactly-once processing semantics requires broker version
0.11.0 or higher.
* If you enable this feature Kafka Streams will use more resources (like
broker connections)
- * compared to the {@link #AT_LEAST_ONCE} case.
+ * compared to {@link #AT_LEAST_ONCE "at_least_once"} and {@link
#EXACTLY_ONCE_V2 "exactly_once_v2"}.
*
- * @see #EXACTLY_ONCE_BETA
+ * @deprecated Since 3.0.0, will be removed in 4.0. Use {@link
#EXACTLY_ONCE_V2 "exactly_once_v2"} instead.
Review comment:
See comment above -- I'm ok with removing this for the
`sendOffsetsToTransaction` since all users have to do there is update one line
of code, but for eos that may require a broker upgrade I'd rather leave this in
to encourage them to do so
--
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:
us...@infra.apache.org
