GitHub user eyalzit opened a pull request:

    https://github.com/apache/spark/pull/16991

    [SPARK-19594][Structured Streaming] StreamingQueryListener fails to handle 
QueryTerminatedEvent if more then one listeners exists

    ## What changes were proposed in this pull request?
    
    currently if multiple streaming queries listeners exists, when a 
QueryTerminatedEvent is triggered, only one of the listeners will be invoked 
while the rest of the listeners will ignore the event.
    this is caused since the the streaming queries listeners bus holds a set of 
running queries ids and when a termination event is triggered, after the first 
listeners is handling the event, the terminated query id is being removed from 
the set. 
    in this PR, the query id will be removed from the set only after all the 
listeners handles the event
    
    ## How was this patch tested?
    
    a test with multiple listeners has been added to StreamingQueryListenerSuite


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/eyalzit/spark master

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/16991.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #16991
    
----
commit ad75ec0eb336f265d638ce97567984a464bae41d
Author: Eyal Zituny <eyal.zit...@equalum.io>
Date:   2017-02-19T10:55:30Z

    [SPARK-19594] StreamingQueryListener fails to handle QueryTerminatedEvent 
if more then one listeners exists

----


---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to