Aitozi commented on issue #4514: [FLINK-7384][cep] Unify event and processing 
time handling in the Abs…
URL: https://github.com/apache/flink/pull/4514#issuecomment-456284959
 
 
   Hi @dawidwys
   I have gone through this PR, I think the unified solution is more clear. 
Maybe better check about the timeCharacteristic in `onTime` because i think we 
just need to `advanceTime` under processingtime and to not have to try to get 
from `elementqueue` . And why this PR was not checked in, Are there sth blocked?

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


With regards,
Apache Git Services

Reply via email to