Address seventh part of 451: Make dataset_id required on Key. #463

dhermes · 2014-12-23T07:38:44Z

Addresses seventh part of #451.

Also requires removing the dataset_id from generated protobufs
due to googleapis/google-cloud-datastore#59. This
occurs also in __key__ filters and ancestor queries.

NOTE: This has #462 as a diffbase.

coveralls · 2014-12-23T07:41:19Z

Coverage remained the same when pulling 0321c41 on dhermes:fix-451-part7 into b343acf on GoogleCloudPlatform:master.

tseaver · 2014-12-30T16:20:16Z

0321c41 comments inline on the commit.

dhermes · 2014-12-30T18:12:07Z

RE: "Mutating key_pb in helper._prepare_key_for_request". It's fine with me, I just didn't want to be presumptuous about callers expectations. If users ever use the public methods on Connection directly, this would cause side effects they may not expect. I also considered adding a flag to Key.to_protobuf() that would allow dropping the dataset_id at creation time.

RE: "Requiring dataset_id in Key constructor". You are right it isn't truly required, but Key instantiation will fail if the implicit one is not set. Should I just add a note about this higher up the docstring?

coveralls · 2014-12-30T18:14:18Z

Coverage remained the same when pulling 0e57202 on dhermes:fix-451-part7 into 8cb41f6 on GoogleCloudPlatform:master.

tseaver · 2014-12-31T21:37:53Z

I'm OK with leaving _prepare_key_for_request as is.

Maybe the :param dataset_id docstring should say, "Required, unless the implicit dataset ID has been set."

gcloud/datastore/test_key.py

Addresses seventh part of googleapis#451. Also requires removing the dataset_id from generated protobufs due to googleapis/google-cloud-datastore#59. This occurs also in __key__ filters and ancestor queries.

coveralls · 2014-12-31T22:31:50Z

Coverage remained the same when pulling 48d2015 on dhermes:fix-451-part7 into e86ba09 on GoogleCloudPlatform:master.

dhermes · 2014-12-31T22:33:19Z

@tseaver PTAL (removing compare_to_proto turned out to make git unhappy on the rebase)

gcloud/datastore/key.py

coveralls · 2014-12-31T22:35:04Z

Coverage remained the same when pulling f697464 on dhermes:fix-451-part7 into e86ba09 on GoogleCloudPlatform:master.

Address seventh part of 451: Make dataset_id required on Key.

… metadata to document_processor_service.proto (#463) * feat: Added Training and Evaluation functions, request, responses and metadata to document_processor_service.proto feat: Added evaluation.proto feat: Added latest_evaluation to processor.proto chore: removed deprecated flag from REPLACE in OperationType in document.proto PiperOrigin-RevId: 511230520 Source-Link: googleapis/googleapis@c53bf8d Source-Link: googleapis/googleapis-gen@5693cbc Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNTY5M2NiYzk4YjY0MTIwODVmZjZkM2FiYTcxOWI2OGUzMDdjNTM1NyJ9 * 🦉 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>

…p/templates/python_library/.kokoro (#463) Source-Link: https://togithub.com/googleapis/synthtool/commit/bb171351c3946d3c3c32e60f5f18cee8c464ec51 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:f62c53736eccb0c4934a3ea9316e0d57696bb49c1a7c86c726e9bb8a2f87dadf

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

Source-Link: googleapis/synthtool@57be0cd Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:ed1f9983d5a935a89fe8085e8bb97d94e41015252c5b6c9771257cf8624367e6 Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com> Co-authored-by: Anthonios Partheniou <[email protected]>

Co-authored-by: Anthonios Partheniou <[email protected]>

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

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

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

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

Co-authored-by: Tim Swast <[email protected]>

* bump google-auth version to 2.13.0 * chore: set minimum to 2.14.1 instead Co-authored-by: Anthonios Partheniou <[email protected]>

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

* feat: use native namespaces instead of pkg_resources * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * Fixed comments in test_packaging.py --------- Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

Add Github workflow to run systests under emulator. Avoid default credential/project lookup under emulator Closes #463.

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

Closes #270

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

dhermes mentioned this pull request Dec 23, 2014

Update Key API to be more user-friendly per design doc discussion #451

Closed

10 tasks

dhermes changed the title ~~Address sixth part of 451: Make dataset_id required on Key.~~ Address seventh part of 451: Make dataset_id required on Key. Dec 30, 2014

dhermes force-pushed the fix-451-part7 branch from 0321c41 to 0e57202 Compare December 30, 2014 18:12

dhermes mentioned this pull request Dec 30, 2014

Address eighth part of 451: Add parent to constructor #476

Merged

dhermes force-pushed the fix-451-part7 branch 2 times, most recently from e1ed6c5 to cc93433 Compare December 31, 2014 22:15

tseaver reviewed Dec 31, 2014
View reviewed changes

gcloud/datastore/test_key.py Outdated

This comment was marked as spam.

Sign in to view

Make dataset_id required on Key.

48d2015

Addresses seventh part of googleapis#451. Also requires removing the dataset_id from generated protobufs due to googleapis/google-cloud-datastore#59. This occurs also in __key__ filters and ancestor queries.

dhermes force-pushed the fix-451-part7 branch from cc93433 to 48d2015 Compare December 31, 2014 22:29

Clarifying required-ness of dataset_id in Key constructor.

f697464

tseaver reviewed Dec 31, 2014
View reviewed changes

gcloud/datastore/key.py Outdated

This comment was marked as spam.

Sign in to view

tseaver added a commit that referenced this pull request Dec 31, 2014

Merge pull request #463 from dhermes/fix-451-part7

8b04561

Address seventh part of 451: Make dataset_id required on Key.

tseaver merged commit 8b04561 into googleapis:master Dec 31, 2014

dhermes deleted the fix-451-part7 branch December 31, 2014 22:38

dhermes added the api: datastore Issues related to the Datastore API. label Dec 31, 2015

parthea pushed a commit that referenced this pull request Aug 15, 2023

chore(main): release 1.13.4 (#463)

b282cf6

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

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

chore(deps): update dependency google-cloud-bigquery to v3.11.0 (#463)

b8f0ada

Co-authored-by: Anthonios Partheniou <[email protected]>

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

chore(deps): update all dependencies (#463)

37d3fa4

Co-authored-by: Anthonios Partheniou <[email protected]>

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

chore(main): release 3.11.0 (#463)

13fc8aa

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

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

chore(deps): update dependency google-cloud-storage to v2.6.0 (#463)

58f8133

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

chore(deps): update all dependencies (#463)

3abd31c

Co-authored-by: Tim Swast <[email protected]>

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

chore(deps): update dependency google-api-core to v1.20.1 (#463)

a62dc8f

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

docs: fix typos in proto/message.py (#463)

fb9ca99

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

fix: require google-auth >= 2.14.1 (#463)

f680f77

* bump google-auth version to 2.13.0 * chore: set minimum to 2.14.1 instead Co-authored-by: Anthonios Partheniou <[email protected]>

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

chore: test minimum dependencies in python 3.7 (#463)

306d199

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

tests: ensure systests pass under emulator in a clean environment (#464)

63ff2c6

Add Github workflow to run systests under emulator. Avoid default credential/project lookup under emulator Closes #463.

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

feat: allow Query.fetch_page for queries with post filters (#463)

1111906

Closes #270

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Address seventh part of 451: Make dataset_id required on Key. #463

Address seventh part of 451: Make dataset_id required on Key. #463

Uh oh!

dhermes commented Dec 23, 2014

Uh oh!

coveralls commented Dec 23, 2014

Uh oh!

tseaver commented Dec 30, 2014

Uh oh!

dhermes commented Dec 30, 2014

Uh oh!

coveralls commented Dec 30, 2014

Uh oh!

tseaver commented Dec 31, 2014

Uh oh!

This comment was marked as spam.

Uh oh!

coveralls commented Dec 31, 2014

Uh oh!

dhermes commented Dec 31, 2014

Uh oh!

This comment was marked as spam.

Uh oh!

coveralls commented Dec 31, 2014

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Address seventh part of 451: Make dataset_id required on Key. #463

Address seventh part of 451: Make dataset_id required on Key. #463

Uh oh!

Conversation

dhermes commented Dec 23, 2014

Uh oh!

coveralls commented Dec 23, 2014

Uh oh!

tseaver commented Dec 30, 2014

Uh oh!

dhermes commented Dec 30, 2014

Uh oh!

coveralls commented Dec 30, 2014

Uh oh!

tseaver commented Dec 31, 2014

Uh oh!

This comment was marked as spam.

Uh oh!

coveralls commented Dec 31, 2014

Uh oh!

dhermes commented Dec 31, 2014

Uh oh!

This comment was marked as spam.

Uh oh!

coveralls commented Dec 31, 2014

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants