rickhofstede commented on a change in pull request #6503: [FLINK-10072] [docs]
Syntax and consistency issues in "The Broadcast State Pattern"
URL: https://github.com/apache/flink/pull/6503#discussion_r210502182
##########
File path: docs/dev/stream/state/broadcast_state.md
##########
@@ -171,9 +171,8 @@ exposes some functionality which is not available to the
`BroadcastProcessFuncti
to register event and/or processing time timers. When a timer fires, the
`onTimer()` (shown above) is invoked with an
`OnTimerContext` which exposes the same functionality as the
`ReadOnlyContext` plus
- the ability to ask if the timer that fired was an event or processing
time one and
- - to query the key associated with the timer.
-
- This is aligned with the `onTimer()` method of the `KeyedProcessFunction`.
+ - to query the key associated with the timer.<br/>
Review comment:
I totally agree, @zentol. To avoid cluttered commits in which we (a) fix
something and (b) improve something, I kept this commit as is (i.e., fixing the
list/rendering issue), and added a second commit in which I remove the
ambiguous line from the text as you proposed. The new commit is
75d815bb4240137016daf19eeb3922aa31da53a1.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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
