You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2019/01/22 06:23:24 UTC

[GitHub] Aitozi removed a comment on issue #7543: [FLINK-10996]Fix the possible state leak with CEP processing time model

Aitozi removed a comment on issue #7543: [FLINK-10996]Fix the possible state leak with CEP processing time model
URL: https://github.com/apache/flink/pull/7543#issuecomment-456284633
 
 
   Hi @dawidwys 
   I have gone through the `FLINK-7384`, I think the unified solution is more clear. Maybe better check about the timeCharacteristic in `onTimer` because i think we just need to `advanceTime` under processingtime and to not have to try to get from `elementqueue` . And why that PR was not checked in, Are there sth blocking? 

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