[GitHub] flink issue #6162: [FLINK-9579][CEP]Remove unneeded clear on elementQueueSta...

2018-06-15 Thread dawidwys 
Github user dawidwys commented on the issue:

https://github.com/apache/flink/pull/6162
  
Yes, will merge shortly.


---

[GitHub] flink issue #6162: [FLINK-9579][CEP]Remove unneeded clear on elementQueueSta...

2018-06-15 Thread Aitozi 
Github user Aitozi commented on the issue:

https://github.com/apache/flink/pull/6162
  
Is this OK? @dawidwys 


---

[GitHub] flink issue #6162: [FLINK-9579][CEP]Remove unneeded clear on elementQueueSta...

2018-06-14 Thread Aitozi 
Github user Aitozi commented on the issue:

https://github.com/apache/flink/pull/6162
  
Thanks for your review , remove the same block in `onProcessingTime`.


---

[GitHub] flink issue #6162: [FLINK-9579][CEP]Remove unneeded clear on elementQueueSta...

2018-06-14 Thread dawidwys 
Github user dawidwys commented on the issue:

https://github.com/apache/flink/pull/6162
  
Actually. Could you also remove the same block from `onProcessingTime`


---

[GitHub] flink issue #6162: [FLINK-9579][CEP]Remove unneeded clear on elementQueueSta...

2018-06-14 Thread dawidwys 
Github user dawidwys commented on the issue:

https://github.com/apache/flink/pull/6162
  
I think this is a valid change. LGTM

Thanks for the contribution!


---