Github user rtudoran commented on the issue:

    https://github.com/apache/flink/pull/3590
  
    @fhueske @sunjincheng121 @hongyuhong @stefanobortoli 
    
    I have run a test to compare the 3 approaches: 
    -windows  based  #3550 
    -processfunction based with events managed in ValueState[Queue] - this PR
    -processfunction based with events managed in MapState[Long,JList]   #3607 
    
    The simple benchmark that I run generates events 1 ms apart (a 5 tuple like 
the one we used in the tests). There are 2 scenarios that I run a simple 
counting over the window contents
    
    Scenario 1)
    
    2 second window (~2000 events in a window) - 100K events  in total generated
    Window based solution:  113839 ms
    Process based (with Queue): 111792 ms
    Process based on MapState: 110533 ms
    
    10 second window (~10000 events in a window) - 200K events  in total 
generated
    Window based solution:  218399ms
    Process based (with Queue): 217343ms
    Process based on MapState: 217657ms
    
    I would say that the approaches are similar in performance (with some small 
advantage for ProcessingFunctions). Regarding the 2 approaches for handing data 
in process windows, I would say that the price to pay for 
serializing/deserializing the whole list of events is matched by 
(serializing/deserializing the timestamp keys + independently deserializing the 
events that need to be removed). Considering that the performance are similar 
personally I believe that the approach with Queue is preferred because we can 
actually gain something (i.e., the order of the events) which will be helpful 
in extending the implementation for full SQL



---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to