GitHub user davies opened a pull request:

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

    [SPARK-16077] [PYSPARK] catch the exception from pickle.whichmodule()

    ## What changes were proposed in this pull request?
    
    In the case that we don't know which module a object came from, will call 
pickle.whichmodule() to go throught all the loaded modules to find the object, 
which could fail because some modules, for example, six, see 
https://bitbucket.org/gutworth/six/issues/63/importing-six-breaks-pickling
    
    We should ignore the exception here, use '__main__' as the module name (it 
means we can't find the module).
    
    ## How was this patch tested?
    
    Manual tests. Can't have a unit test for this.


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

    $ git pull https://github.com/davies/spark whichmodule

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

    https://github.com/apache/spark/pull/13788.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 #13788
    
----
commit 0d3df9a627b139d9d8d715f96a4fdd655a31c8b8
Author: Davies Liu <[email protected]>
Date:   2016-06-20T20:53:31Z

    catch the exception from pickle.whichmodule()

----


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

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to