[GitHub] yanghua commented on a change in pull request #6503: [FLINK-10072] [docs] Syntax and consistency issues in "The Broadcast State Pattern"

[GitBox <gi...@apache.org>]

yanghua 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_r207944485
 
 

 ##########
 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:
   It seems the old style is OK, from here: https://ci.apache.org/projects/flink/flink-docs-master/dev/stream/state/broadcast_state.html , it can break line without "<br />".

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services