davidradl commented on code in PR #26884:
URL: https://github.com/apache/flink/pull/26884#discussion_r2279278917
##########
flink-core/src/main/java/org/apache/flink/api/common/eventtime/WatermarkStrategy.java:
##########
@@ -223,8 +223,8 @@ static <T> WatermarkStrategy<T> forMonotonousTimestamps() {
/**
* Creates a watermark strategy for situations where records are out of
order, but you can place
* an upper bound on how far the events are out of order. An out-of-order
bound B means that
- * once the an event with timestamp T was encountered, no events older
than {@code T - B} will
- * follow any more.
+ * once an event with timestamp T was encountered, no events older than
{@code T - B} will
+ * follow anymore.
Review Comment:
This text seems to be almost an incorrect copy of the [class javadoc
](https://nightlies.apache.org/flink/flink-docs-release-1.20/api/java/org/apache/flink/api/common/eventtime/BoundedOutOfOrdernessWatermarks.html).
I think a link to the class javadoc, with the existing `@see` or similar and
then javadoc including the `@param` would seem more appropriate. Maybe
something like
`Creates a watermark strategy for @see BoundedOutOfOrdernessWatermarks using
the supplied out-of-order bound`
@param maxOutOfOrderness out-of-order bound duration.
--
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]
