Skip to content

Properly handling entity meanings in datastore. #1241

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
dhermes merged 1 commit into from
Dec 21, 2015
Merged

Properly handling entity meanings in datastore. #1241

dhermes merged 1 commit into from
Dec 21, 2015

Conversation

@dhermes
Copy link
Contributor

@dhermes dhermes commented Nov 24, 2015

To do this, added entity_to_protobuf method that could be used recursively. Also solves #1206 since recursively serializing nested entities to protobuf was being done incorrectly.

Fixes #1065. Fixes #1206.

@dhermes dhermes added the api: datastore Issues related to the Datastore API. label Nov 24, 2015
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 24, 2015
@dhermes
Copy link
Contributor Author

dhermes commented Nov 24, 2015

@tseaver I think this spells the end of inheriting from dict.

dhermes
dhermes reviewed Nov 24, 2015
View reviewed changes
gcloud/datastore/helpers.py

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@dhermes
Copy link
Contributor Author

dhermes commented Dec 1, 2015

@tseaver Bump

2 similar comments
@dhermes
Copy link
Contributor Author

dhermes commented Dec 9, 2015

@tseaver Bump

@dhermes
Copy link
Contributor Author

dhermes commented Dec 16, 2015

@tseaver Bump

@dhermes dhermes mentioned this pull request Dec 17, 2015
Merged
@dhermes
Copy link
Contributor Author

dhermes commented Dec 17, 2015

@tseaver It is now merge-able

tseaver
tseaver reviewed Dec 18, 2015
View reviewed changes
gcloud/datastore/helpers.py Outdated

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

@dhermes
Properly handling entity meanings in datastore.
5873266 
To do this, added entity_to_protobuf method that could be used
recursively. Also solves googleapis#1206 since recursively serializing
nested entities to protobuf was being done incorrectly.

Fixes googleapis#1065. Fixes googleapis#1206.
@dhermes
Copy link
Contributor Author

dhermes commented Dec 19, 2015

@tseaver PTAL, I went with the simpler version of _get_meaning and we should be all set.

@tseaver
Copy link
Contributor

tseaver commented Dec 21, 2015

LGTM

dhermes added a commit that referenced this pull request Dec 21, 2015
@dhermes
Merge pull request #1241 from dhermes/store-meanings
0870be4 
Properly handling entity meanings in datastore.
@dhermes dhermes merged commit 0870be4 into googleapis:master Dec 21, 2015
@dhermes dhermes deleted the store-meanings branch December 21, 2015 21:10
@dhermes dhermes mentioned this pull request Dec 22, 2015
Closed
@dhermes dhermes mentioned this pull request Jan 25, 2016
Merged
@dhermes dhermes mentioned this pull request Mar 23, 2016
Closed
parthea pushed a commit that referenced this pull request Sep 22, 2023
@lukesneeringer @danoscarmike
Do not base64 encode the input content. [(#1241)](GoogleCloudPlatform…
7b4fb62 
…/python-docs-samples#1241)

Protobuf handles the base64 encoding for you, and so the content was being base64 encoded twice.
parthea pushed a commit that referenced this pull request Nov 24, 2025
@renovate-bot
chore(deps): update dependency googleapis-common-protos to v1.56.0 (#…
709101d 
…1241)
parthea pushed a commit that referenced this pull request Nov 26, 2025
@arithmetic1728
fix: don't retry if error or error_description is not string (#1241)
633a84c 
* fix: don't retry if error or error_description is not string

* chore: fix flaky sample test

* chore: update sys test cred
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

api: datastore Issues related to the Datastore API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@dhermes @tseaver @googlebot