Skip to content

fix: disuse __slots__ in most places#330

Merged
chrisrossi merged 1 commit intogoogleapis:masterfrom
chrisrossi:fix-311
Feb 11, 2020
Merged

fix: disuse __slots__ in most places#330
chrisrossi merged 1 commit intogoogleapis:masterfrom
chrisrossi:fix-311

Conversation

@chrisrossi
Copy link
Contributor

In Python 2.7, classes which use __slots__ can't be pickled. Users
have reported problems trying to pickle NDB entities, indicating there
is some perceived use case for pickling entities.

Fixes #311

In Python 2.7, classes which use `__slots__` can't be pickled. Users
have reported problems trying to pickle NDB entities, indicating there
is some perceived use case for pickling entities.

Fixes googleapis#311
@chrisrossi chrisrossi requested a review from cguardia February 11, 2020 16:14
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 11, 2020
@chrisrossi
Copy link
Contributor Author

chrisrossi commented Feb 11, 2020

Kokoro had a transient error related to intersphinx:

failed to reach any of the inventories with the following issues:
intersphinx inventory 'https://grpc.io/grpc/python/objects.inv' not fetchable due to <class 'requests.exceptions.ConnectionError'>: ('Connection aborted.', ConnectionResetError(104, 'Connection reset by peer'))
Command sphinx-build -W -T -N -b html -d docs/_build/doctrees/ docs/ docs/_build/html/ failed with exit code 2

@chrisrossi chrisrossi added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 11, 2020
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Feb 11, 2020
Copy link
Contributor

@cguardia cguardia left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thanks.

@chrisrossi chrisrossi merged commit a8b723b into googleapis:master Feb 11, 2020
@chrisrossi chrisrossi deleted the fix-311 branch February 11, 2020 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Pickling problematic with Python 2

4 participants