Github user Aitozi commented on the issue:
https://github.com/apache/flink/pull/6224
Get it, this change indeed will lead to the situation you mentioned.
> What are the cases where this change leads to an improvement
I just read code and think the order is not suitable.
> I think it rarely happens that a timer fires while a window is empty
I think here is not the one in fact, because we get the window content each
time before get the trigger result. AFAIK in there is `cleanupTimer` and
`fireTimer` in `internalTimerService` in `windowOperator`. we only have to get
window content for `fireTimer` of a window. In now implementation, we have to
extract both for the `cleanupTimer` and `fireTimer`.
---
