apache / apache/beam

Test non default pickle library

Open
#21,282 0 comments 0 reactions 0 assignees View on GitHub
core improvement P3 python
Dominant language
Java
Stars
8.7k
Forks
4.7k
Avg merge
1d 20h
Merged PRs (30d)
196

Description

The python SDK recently added support for the cloudpickle library though the default is still dill. See [_https://s.apache.org/beam-picklers_](_https://s.apache.org/beam-picklers_) and https://issues.apache.org/jira/browse/BEAM-8123.

 

We intend to switch this over to cloudpickle as the default.

 

We should have a suite of E2E tests to make sure we don't introduce regressions into whichever pickle library is not the default.  That is to say:

First -  E2E tests for the using the cloudpickle library.

After cloudpickle library becomes the default - switch these E2E tests to cover dill.

 

This will allow test coverage while we hopefully transition to just supporting a single library.

Imported from Jira [BEAM-13385](https://issues.apache.org/jira/browse/BEAM-13385). Original Jira may contain additional context.
Reported by: Ryan.Thompson.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.