Support entity protobufs #232

tseaver · 2014-10-09T14:19:40Z

No description provided.

tseaver · 2014-10-09T14:44:11Z

Hold of merging: this PR should really deal with marshalling entity_value in Connection.save_entity and Query.filter as well.

dhermes · 2014-10-09T14:55:35Z

Should I hold off review also until the code is in place?

Handles compount fields (entity, key) not assignable via setattr.

tseaver · 2014-10-09T15:41:59Z

@dhermes Should be OK to review now.

tseaver · 2014-10-09T15:46:03Z

Hmm, with 'helpers.set_protobuf_value()' in place, I think we should deprecate (as an API) 'helpers.get_protobuf_name_and_value()': anybody using the latter is going to run into the same issues with composite values (keys, entities, later lists).

gcloud/datastore/helpers.py

dhermes · 2014-10-09T18:08:59Z

RE: helpers.get_protobuf_name_and_value() I agree. Let's reduce the API surface at every chance.

@dhermes

Incorporates feedback from @dhermes.

Entities are not required to have datasets w/ active connections.

tseaver · 2014-10-09T19:54:05Z

I'm not sure anything in the 'helpers' module should be an API at all: maybe we should rename them with leading underscores to highlight that fact?

dhermes · 2014-10-09T20:06:05Z

SGTM. Since this PR touches everything in the module I don't think it's necessary to break out a new PR for the renaming.

@dhermes

Incorporates feedback from @dhermes.

Signal that they are *not* part of the gcloud-python API.

Note that these are functions, not methods.

dhermes · 2014-10-09T20:38:56Z

LGTM pending Travis.

Change that might be worth it: renaming helpers module as _helpers.

Your call either way.

Further emphasize non-API nature.

fix(deps): require proto-plus>=1.15.0

Source-Link: googleapis/synthtool@d730883 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:4370ced27a324687ede5da07132dcdc5381993502a5e8a3e31e16dc631d026f0 Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* docs: point to Search Console for domain ownership verification PiperOrigin-RevId: 521027761 Source-Link: googleapis/googleapis@e6b9d2d Source-Link: googleapis/googleapis-gen@6ae4a2c Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNmFlNGEyY2QwNGFmYzAxMjI0MzQ2YjM5YzE5YTNjYjRlZmE0M2ZiMyJ9 * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

Source-Link: googleapis/synthtool@e44dc0c Post-Processor: gcr.io/repo-automation-bots/owlbot-python:latest@sha256:5ff7446edeaede81c3ed58b23a4e76a5403fba1350ce28478045657303b6479d

- [ ] Regenerate this pull request now. fix: resolve DuplicateCredentialArgs error when using credentials_file committer: parthea PiperOrigin-RevId: 425964861 Source-Link: googleapis/googleapis@84b1a5a Source-Link: googleapis/googleapis-gen@4fb761b Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNGZiNzYxYmJkODUwNmFjMTU2ZjQ5YmFjNWYxODMwNmFhOGViM2FhOCJ9

Source-Link: googleapis/synthtool@f15cc72 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:bc5eed3804aec2f05fad42aacf973821d9500c174015341f721a984a0825b6fd

) Source-Link: https://togithub.com/googleapis/synthtool/commit/0c7b0333f44b2b7075447f43a121a12d15a7b76a Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:08e34975760f002746b1d8c86fdc90660be45945ee6d9db914d1508acdf9a547

* chore: update copyright year to 2022 PiperOrigin-RevId: 431037888 Source-Link: googleapis/googleapis@b3397f5 Source-Link: googleapis/googleapis-gen@510b54e Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNTEwYjU0ZTFjZGVmZDUzMTczOTg0ZGYxNjY0NTA4MTMwOGZlODk3ZSJ9 * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* docs: Fix formatting of request arg in docstring chore: Update gapic-generator-python to v1.9.1 PiperOrigin-RevId: 518604533 Source-Link: googleapis/googleapis@8a085ae Source-Link: googleapis/googleapis-gen@b2ab4b0 Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiYjJhYjRiMGEwYWUyOTA3ZTgxMmMyMDkxOThhNzRlMDg5OGFmY2IwNCJ9 * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

…pprove] (#232) Source-Link: https://togithub.com/googleapis/synthtool/commit/e3a1277ac35fc88c09db1930533e24292b132ced Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:452901c74a22f9b9a3bd02bce780b8e8805c97270d424684bff809ce5be8c2a2

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

Use CircleCI for testing pandas-gbq. This configuration uses the canonical Python and miniconda3 Docker images to reduce problems caused by a unique configuration. Also: * Re-enables conda tests. * Adds Python 3.7 tests.

* chore(python): Add Python 3.12 Source-Link: googleapis/synthtool@af16e6d Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:bacc3af03bff793a03add584537b36b5644342931ad989e3ba1171d3bd5399f5 * Add constraints file for python 3.12 * Update trove classifier for python 3.12 --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> Co-authored-by: Anthonios Partheniou <[email protected]>

…ct' method (#232) Solution to the following issue #231, should work as boolean value and according to the value the GoogleAdsRow default columns should or should not return in results. If including_default_value_fields is False default columns should not return and if True default values should return now. Closes #231.

…le (#232) Source-Link: googleapis/synthtool@a7ed11e Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:6e7328583be8edd3ba8f35311c76a1ecbc823010279ccb6ab46b7a76e25eafcc Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* chore(deps): update all dependencies to v4 * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* feat: Introduce compatibility with native namespace packages * Add missing files * lint

* chore: use gapic-generator-python 0.63.4 chore: fix snippet region tag format chore: fix docstring code block formatting PiperOrigin-RevId: 430730865 Source-Link: googleapis/googleapis@ea58002 Source-Link: googleapis/googleapis-gen@ca893ff Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiY2E4OTNmZjhhZjI1ZmM3ZmUwMDFkZTE0MDVhNTE3ZDgwNDQ2ZWNjYSJ9 * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * chore: delete duplicates Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> Co-authored-by: Bu Sun Kim <[email protected]>

Co-authored-by: Daniel Sanche <[email protected]>

* Release v0.2.0

Co-authored-by: Daniel Sanche <[email protected]>

tseaver added 2 commits October 9, 2014 10:16

Add explicit support for 'entity_value' in protobufs.

1a79c89

Lint removal.

e5954fc

beriberikix added the in progress label Oct 9, 2014

tseaver mentioned this pull request Oct 9, 2014

Support 'blob_value' / 'blob_key_value' in datastore protobufs #233

Closed

tseaver added api: datastore Issues related to the Datastore API. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. labels Oct 9, 2014

tseaver mentioned this pull request Oct 9, 2014

Add Support for Query Cursor & Order #126

Closed

tseaver closed this Oct 9, 2014

beriberikix reopened this Oct 9, 2014

beriberikix added in progress and removed in progress labels Oct 9, 2014

tseaver added 2 commits October 9, 2014 11:30

Add helpers.set_protobuf_value.

c8f9a7a

Handles compount fields (entity, key) not assignable via setattr.

Use helpers.set_protobuf_value to support composite types.

37304c9

dhermes reviewed Oct 9, 2014
View reviewed changes

gcloud/datastore/helpers.py Outdated

This comment was marked as spam.

Sign in to view

tseaver added 2 commits October 9, 2014 15:48

Fix docstring param name.

f5ee0d7

Incorporates feedback from @dhermes.

Drop XXX comment.

8fe4168

Entities are not required to have datasets w/ active connections.

tseaver added 3 commits October 9, 2014 16:28

Don't set default key on entity protobuf if entity has no key.

1679154

Incorporates feedback from @dhermes.

Rename helper functions with leading underscore.

49f04b5

Signal that they are *not* part of the gcloud-python API.

Update doctring to indicate that helper functions are not API.

275f6bd

Note that these are functions, not methods.

Rename datastore.helpers -> _helpers.

fef83a1

Further emphasize non-API nature.

parthea pushed a commit that referenced this pull request Sep 22, 2023

chore(deps): update dependency google-cloud-bigquery to v2.24.0 (#232)

e446c1d

parthea pushed a commit that referenced this pull request Sep 22, 2023

chore(deps): update dependency pytest to v7.1.2 (#232)

c7fef3e

parthea added a commit that referenced this pull request Sep 22, 2023

fix(deps): require google-api-core>=1.31.5, >=2.3.2 (#232)

496af7f

fix(deps): require proto-plus>=1.15.0

parthea pushed a commit that referenced this pull request Sep 22, 2023

chore: use python-samples-reviewers (#232)

594c4da

parthea pushed a commit that referenced this pull request Oct 21, 2023

chore(deps): update dependency google-cloud-monitoring to v2.6.0 (#232)

28a815e

parthea added a commit that referenced this pull request Oct 22, 2023

chore: update python-samples-owners as samples codeowner (#232)

721de7a

parthea pushed a commit that referenced this pull request Jan 10, 2025

chore(main): release 1.8.3 (#232)

7e023cb

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

parthea pushed a commit that referenced this pull request Aug 21, 2025

chore(deps): update dependency google-cloud-bigquery to v2.21.0 (#232)

3d0849e

parthea pushed a commit that referenced this pull request Sep 16, 2025

chore(deps): update dependency google-cloud-bigquery to v2.21.0 (#232)

bf22278

parthea added a commit that referenced this pull request Nov 24, 2025

feat: Introduce compatibility with native namespace packages (#232)

5fe1506

* feat: Introduce compatibility with native namespace packages * Add missing files * lint

parthea pushed a commit that referenced this pull request Nov 24, 2025

chore(deps): update dependency google-cloud-bigquery to v3.15.0 (#232)

1fd7574

parthea pushed a commit that referenced this pull request Nov 24, 2025

chore(deps): update dependency google-cloud-bigquery to v2.13.1 (#232)

bd643a5

Co-authored-by: Daniel Sanche <[email protected]>

parthea pushed a commit that referenced this pull request Nov 24, 2025

chore(release): Release v0.2.0 (#232)

66bc947

* Release v0.2.0

parthea pushed a commit that referenced this pull request Nov 25, 2025

chore(deps): update dependency google-cloud-bigquery to v2.13.1 (#232)

28cc0f9

Co-authored-by: Daniel Sanche <[email protected]>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Support entity protobufs #232

Support entity protobufs #232

Uh oh!

tseaver commented Oct 9, 2014

Uh oh!

tseaver commented Oct 9, 2014

Uh oh!

dhermes commented Oct 9, 2014

Uh oh!

tseaver commented Oct 9, 2014

Uh oh!

tseaver commented Oct 9, 2014

Uh oh!

This comment was marked as spam.

Uh oh!

dhermes commented Oct 9, 2014

Uh oh!

tseaver commented Oct 9, 2014

Uh oh!

dhermes commented Oct 9, 2014

Uh oh!

dhermes commented Oct 9, 2014

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Support entity protobufs #232

Support entity protobufs #232

Uh oh!

Conversation

tseaver commented Oct 9, 2014

Uh oh!

tseaver commented Oct 9, 2014

Uh oh!

dhermes commented Oct 9, 2014

Uh oh!

tseaver commented Oct 9, 2014

Uh oh!

tseaver commented Oct 9, 2014

Uh oh!

This comment was marked as spam.

Uh oh!

dhermes commented Oct 9, 2014

Uh oh!

tseaver commented Oct 9, 2014

Uh oh!

dhermes commented Oct 9, 2014

Uh oh!

dhermes commented Oct 9, 2014

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants