Adding extra parameters for querying Cloud Storage keys. by dhermes · Pull Request #332 · googleapis/google-cloud-python

dhermes · 2014-11-03T22:07:11Z

Will implement your suggestion in a 2nd commit.

Also notice in #319 that there should be an easier way to pass in extra_params than to manually create a _KeyIterator.

Sign in to view

coveralls · 2014-11-03T22:24:37Z

Changes Unknown when pulling cf1f2a9 on dhermes:add-params-to-key-iterator into * on GoogleCloudPlatform:master*.

coveralls · 2014-11-03T22:39:36Z

Coverage remained the same when pulling f6d9aee on dhermes:add-params-to-key-iterator into 2b73c7b on GoogleCloudPlatform:master.

Sign in to view

tseaver · 2014-11-04T17:55:30Z

LGTM beyond my clarity quibble.

coveralls · 2014-11-04T18:22:26Z

Coverage remained the same when pulling 062bba5 on dhermes:add-params-to-key-iterator into abe3d7c on GoogleCloudPlatform:master.

coveralls · 2014-11-04T18:43:32Z

Coverage remained the same when pulling 161ecbc on dhermes:add-params-to-key-iterator into a410c6a on GoogleCloudPlatform:master.

coveralls · 2014-11-04T21:50:55Z

Coverage remained the same when pulling 6851159 on dhermes:add-params-to-key-iterator into a410c6a on GoogleCloudPlatform:master.

Sign in to view

Also making sure tests pass by using the `upload_` prefix instead of the `set_contents_` prefix.

dhermes · 2014-11-04T22:14:04Z

@tseaver Updated docstring (my bad) and rebased. PTAL

tseaver · 2014-11-04T22:18:57Z

LGTM.

coveralls · 2014-11-04T22:21:11Z

Coverage remained the same when pulling 2f28e9f on dhermes:add-params-to-key-iterator into aa22386 on GoogleCloudPlatform:master.

Adding extra parameters for querying Cloud Storage keys.

…rameters. See github issue googleapis#332

- [ ] Regenerate this pull request now. PiperOrigin-RevId: 451250442 Source-Link: googleapis/googleapis@cca5e81 Source-Link: googleapis/googleapis-gen@0b219da Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMGIyMTlkYTE2MWE4YmRjYzNjNmY3YjJlZmNkODIxMDUxODJhMzBjYSJ9

…1.11.0 (#332)

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

* chore(deps): update all dependencies * revert update for google-auth * add pin for google-cloud-storage for py < 3.7 Co-authored-by: Anthonios Partheniou <partheniou@google.com>

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

* chore: update region tags to use correct product prefix * chore: update region tags to use correct product prefix * chore: update region tags to use correct product prefix

* chore: Update gapic-generator-python to v1.8.4 PiperOrigin-RevId: 507808936 Source-Link: googleapis/googleapis@64cf849 Source-Link: googleapis/googleapis-gen@53c48ca Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNTNjNDhjYWMxNTNkM2IzN2YzZDJjMmRlYzQ4MzBjZmQ5MWVjNDE1MyJ9 * 🦉 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@50db768 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:e09366bdf0fd9c8976592988390b24d53583dd9f002d476934da43725adbb978 Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

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

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

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

* Update api_shortname in .repo-metadata.json * chore: Update api_shortname in .repo-metadata.json * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * 🦉 Updates from OwlBot See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * fix replacement in owlbot.py * fix indent * restore coverage at 100% Co-authored-by: Anthonios Partheniou <partheniou@google.com> Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

Co-authored-by: Anthonios Partheniou <partheniou@google.com>

Source-Link: https://togithub.com/googleapis/synthtool/commit/92006bb3cdc84677aa93c7f5235424ec2b157146 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:2e247c7bf5154df7f98cce087a20ca7605e236340c7d6d1a14447e5c06791bd6

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

* chore(deps): update all dependencies * remove pin for python 3.6 Co-authored-by: Anthonios Partheniou <partheniou@google.com>

* docs: Minor formatting chore: Update gapic-generator-python to v1.11.5 build: Update rules_python to 0.24.0 PiperOrigin-RevId: 563436317 Source-Link: googleapis/googleapis@42fd37b Source-Link: googleapis/googleapis-gen@280264c Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMjgwMjY0Y2EwMmZiOTMxNmI0MjM3YTk2ZDBhZjFhMjM0M2E4MWE1NiJ9 * 🦉 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@b0eb8a8 Post-Processor: gcr.io/repo-automation-bots/owlbot-python:latest@sha256:df50e8d462f86d6bcb42f27ecad55bb12c404f1c65de9c6fe4c4d25120080bd6

Flattened method fields generate names in client methods. Barring a central authority, any source of names generates the possibility of name collisions and requires disambiguation. Before this fix, flattened fields could generate name collisions with imported modules. This commit adds field names to the context with which names are disambiguated and subjects LRO operation info structures to the same collision avoidance logic as other rich data types. Unblocks autogenerated unit tests for Datalabeling API.

* fix: fixes bug in the test.

Also, include materialized views in 'get_view_names'. Closes #332.

* chore(python): drop python 3.6 Source-Link: googleapis/synthtool@4f89b13 Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:e7bb19d47c13839fe8c147e50e02e8b6cf5da8edd1af8b82208cd6f66cc2829c * add api_description to .repo-metadata.json * require python 3.7+ in setup.py * remove python 3.6 sample configs * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * remove python 3.6 from noxfile * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * remove python 3.6 from noxfile * 🦉 Updates from OwlBot post-processor See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md * ci: update replacement in owlbot.py * 🦉 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> Co-authored-by: Anthonios Partheniou <partheniou@google.com>

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

- Added support for Custom Classifier entities - Added Unit Tests for Classifier output - Added input validation for `Document.split_pdf()` Fixes #332 🦕

fixes #332

Closes #332

* fix: ignore cross reference conversion in examples * test: update unit test

dhermes mentioned this pull request Nov 3, 2014

Implementing storage regression tests to match gcloud-node. #319

Merged

tseaver reviewed Nov 3, 2014
View reviewed changes

Comment thread gcloud/storage/iterator.py Outdated

This comment was marked as spam.

Sign in to view

This comment was marked as spam.

Sign in to view

dhermes force-pushed the add-params-to-key-iterator branch from cf1f2a9 to f6d9aee Compare November 3, 2014 22:31

dhermes added api: storage Issues related to the Cloud Storage API. type: feature request ‘Nice-to-have’ improvement, new feature or different behavior or design. labels Nov 3, 2014

tseaver reviewed Nov 4, 2014
View reviewed changes

Comment thread gcloud/storage/iterator.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 force-pushed the add-params-to-key-iterator branch from f6d9aee to 062bba5 Compare November 4, 2014 18:16

dhermes force-pushed the add-params-to-key-iterator branch from 062bba5 to 161ecbc Compare November 4, 2014 18:37

tseaver reviewed Nov 4, 2014
View reviewed changes

Comment thread gcloud/storage/iterator.py Outdated

This comment was marked as spam.

Sign in to view

dhermes added 4 commits November 4, 2014 14:13

Adding extra parameters for querying Cloud Storage keys.

029249c

Testing interaction with page token and extra params in iterator.

88e5234

Also making sure tests pass by using the `upload_` prefix instead of the `set_contents_` prefix.

Using {} as default in iterator for extra_params and get_query_params.

018795f

Updating docstring in Iterator.get_query_params.

2f28e9f

dhermes force-pushed the add-params-to-key-iterator branch from 6851159 to 2f28e9f Compare November 4, 2014 22:13

dhermes added a commit that referenced this pull request Nov 4, 2014

Merge pull request #332 from dhermes/add-params-to-key-iterator

826ea69

Adding extra parameters for querying Cloud Storage keys.

dhermes merged commit 826ea69 into googleapis:master Nov 4, 2014

dhermes deleted the add-params-to-key-iterator branch November 4, 2014 22:24

urshala pushed a commit to urshala/google-cloud-python that referenced this pull request Jan 17, 2020

fixed missing parentheses which throws an error when using default pa…

537cc99

…rameters. See github issue googleapis#332

parthea pushed a commit that referenced this pull request Jun 4, 2023

chore(deps): update dependency google-cloud-recaptcha-enterprise to v…

c43c843

…1.11.0 (#332)

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

chore(main): release 2.9.2 (#332)

f463a2c

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

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

chore(deps): update dependency google-cloud-vision to v2.7.2 (#332)

3105b40

Co-authored-by: Anthonios Partheniou <partheniou@google.com>

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

chore(main): release 0.16.1 (#332)

d10dee2

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

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

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

310c508

* chore(deps): update all dependencies * remove pin for python 3.6 Co-authored-by: Anthonios Partheniou <partheniou@google.com>

release-please Bot mentioned this pull request Oct 30, 2023

chore: release main #11913

Merged

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

fix: fixes bug in the test. (#332)

436ab73

* fix: fixes bug in the test.

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

docs: fix typo in library name (#332)

acc38f1

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

fix: include external tables in 'get_table_names' (#363)

d979766

Also, include materialized views in 'get_view_names'. Closes #332.

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

fix: remove setup.cfg configuration for creating universal wheels (#332)

2d915b7

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

chore(main): release 1.4.3 (#332)

75ec6de

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

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

fix: Add support for Classifier entities (#333)

9cc07cc

- Added support for Custom Classifier entities - Added Unit Tests for Classifier output - Added input validation for `Document.split_pdf()` Fixes #332 🦕

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

chore(deps): update dependency google-cloud-pubsub to v2.6.1 (#332)

81dd70d

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

chore(deps): update dependency google-cloud-pubsub to v2.6.1 (#332)

5d30275

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

Fix typo in jwt docs. (#332)

c6a34a3

parthea pushed a commit that referenced this pull request Mar 6, 2026

django: fix modulo operator

1d78f64

fixes #332

parthea pushed a commit that referenced this pull request Mar 9, 2026

fix: avoid triggering global logging config (#333)

22a0166

Closes #332

parthea pushed a commit that referenced this pull request Mar 9, 2026

fix: ignore cross reference conversion in examples (#332)

7ab4cba

* fix: ignore cross reference conversion in examples * test: update unit test

Uh oh!

Conversation

dhermes commented Nov 3, 2014

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

coveralls commented Nov 3, 2014

Uh oh!

coveralls commented Nov 3, 2014

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

This comment was marked as spam.

Uh oh!

tseaver commented Nov 4, 2014

Uh oh!

coveralls commented Nov 4, 2014

Uh oh!

coveralls commented Nov 4, 2014

Uh oh!

coveralls commented Nov 4, 2014

Uh oh!

This comment was marked as spam.

Uh oh!

dhermes commented Nov 4, 2014

Uh oh!

tseaver commented Nov 4, 2014

Uh oh!

coveralls commented Nov 4, 2014

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants