Support entity protobufs by tseaver · Pull Request #232 · googleapis/google-cloud-python

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).

Sign in to view

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.

)

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

…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>

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

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

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

[![WhiteSource Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Change | Age | Adoption | Passing | Confidence | |---|---|---|---|---|---| | [google-cloud-storage](https://togithub.com/googleapis/python-storage) | `==1.42.1` -> `==1.42.2` | [![age](https://badges.renovateapi.com/packages/pypi/google-cloud-storage/1.42.2/age-slim)](https://docs.renovatebot.com/merge-confidence/) | [![adoption](https://badges.renovateapi.com/packages/pypi/google-cloud-storage/1.42.2/adoption-slim)](https://docs.renovatebot.com/merge-confidence/) | [![passing](https://badges.renovateapi.com/packages/pypi/google-cloud-storage/1.42.2/compatibility-slim/1.42.1)](https://docs.renovatebot.com/merge-confidence/) | [![confidence](https://badges.renovateapi.com/packages/pypi/google-cloud-storage/1.42.2/confidence-slim/1.42.1)](https://docs.renovatebot.com/merge-confidence/) | --- ### Release Notes <details> <summary>googleapis/python-storage</summary> ### [`v1.42.2`](https://togithub.com/googleapis/python-storage/blob/master/CHANGELOG.md#1422-httpswwwgithubcomgoogleapispython-storagecomparev1421v1422-2021-09-16) [Compare Source](https://togithub.com/googleapis/python-storage/compare/v1.42.1...v1.42.2) </details> --- ### Configuration 📅 **Schedule**: At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] If you want to rebase/retry this PR, check this box. --- This PR has been generated by [WhiteSource Renovate](https://renovate.whitesourcesoftware.com). View repository job log [here](https://app.renovatebot.com/dashboard#github/googleapis/python-vision).

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 <partheniou@google.com>

…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>

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

Comment threadgcloud/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 20, 2023
chore(deps): update dependency google-api-python-client to v2.60.0 (#232
d9f3f1c
)

parthea pushed a commit that referenced this pull request Sep 22, 2023
chore(main): release 1.6.0 (#232)
aff85e8
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

parthea pushed a commit that referenced this pull request Sep 22, 2023
chore(deps): update dependency pytest to v7.1.2 (#232)
a026d76

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

Navigation Menu

Uh oh!

Support entity protobufs - #232

Support entity protobufs#232
tseaver merged 10 commits into
googleapis:masterfrom
tseaver:support-entity-protobufs

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

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