GitHub user tdas opened a pull request:

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

    Fixed state store bug

    ## What changes were proposed in this pull request?
    
    StateStore.get() causes temporary files to be created immediately, even if 
the store is not used to make updates for new version. The temp file is not 
closed as store.commit() is not called in those cases, thus keeping the output 
stream to temp file open forever.
    
    This PR fixes it by opening the temp file only when there are updates being 
made.
    
    ## How was this patch tested?
    
    New unit test
    


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

    $ git pull https://github.com/tdas/spark SPARK-18416

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

    https://github.com/apache/spark/pull/15859.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 #15859
    
----
commit a9d8edf6224c25fe9b4d016cfc85d0f1d7dcfa08
Author: Tathagata Das <tathagata.das1...@gmail.com>
Date:   2016-11-12T01:00:15Z

    Fixed state store bug

----


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