GitHub user rgbkrk opened a pull request:
https://github.com/apache/spark/pull/18282
[SPARK-21070][PYSPARK] Upgrade cloudpickle
## What changes were proposed in this pull request?
This brings in fixes and upgrades from the
[cloudpickle](https://github.com/cloudpipe/cloudpickle) module. Notable fixes:
* Import submodules accessed by pickled functions
(https://github.com/cloudpipe/cloudpickle/pull/80)
* Support recursive functions inside closures
(https://github.com/cloudpipe/cloudpickle/pull/89,
https://github.com/cloudpipe/cloudpickle/pull/90)
* Fix ResourceWarnings and DeprecationWarnings
(https://github.com/cloudpipe/cloudpickle/pull/88)
* Assume modules with __file__ attribute are not dynamic
(https://github.com/cloudpipe/cloudpickle/pull/85)
* Make cloudpickle Python 3.6 compatible
(https://github.com/cloudpipe/cloudpickle/pull/72)
* Allow pickling of builtin methods
(https://github.com/cloudpipe/cloudpickle/pull/57)
* Add ability to pickle dynamically created modules
(https://github.com/cloudpipe/cloudpickle/pull/52)
* Support method descriptor
(https://github.com/cloudpipe/cloudpickle/pull/46)
* No more pickling of closed files, was broken on Python 3
(https://github.com/cloudpipe/cloudpickle/pull/32)
## How was this patch tested?
The [test suite for
cloudpickle](https://github.com/cloudpipe/cloudpickle/tree/master/tests) tests
the internal implementation. There are integration tests here in PySpark that
use the `CloudPickleSerializer`.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/rgbkrk/spark cloudpickle-upgrades
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/spark/pull/18282.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 #18282
----
commit b84222b6ea660ce5ec1aedfee50e297b436eb824
Author: Kyle Kelley <rgb...@gmail.com>
Date: 2017-06-12T22:19:29Z
[SPARK-21070][PYSPARK] Upgrade cloudpickle
This brings in fixes and upgrades from the
[cloudpickle](https://github.com/cloudpipe/cloudpickle) module, notably:
* Import submodules accessed by pickled functions
(https://github.com/cloudpipe/cloudpickle/pull/80)
* Support recursive functions inside closures
(https://github.com/cloudpipe/cloudpickle/pull/89,
https://github.com/cloudpipe/cloudpickle/pull/90)
* Fix ResourceWarnings and DeprecationWarnings
(https://github.com/cloudpipe/cloudpickle/pull/88)
* Assume modules with __file__ attribute are not dynamic
(https://github.com/cloudpipe/cloudpickle/pull/85)
* Make cloudpickle Python 3.6 compatible
(https://github.com/cloudpipe/cloudpickle/pull/72)
* Allow pickling of builtin methods
(https://github.com/cloudpipe/cloudpickle/pull/57)
* Add ability to pickle dynamically created modules
(https://github.com/cloudpipe/cloudpickle/pull/52)
* Support method descriptor
(https://github.com/cloudpipe/cloudpickle/pull/46)
* No more pickling of closed files, was broken on Python 3
(https://github.com/cloudpipe/cloudpickle/pull/32)
----
---
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
