FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Using 'fields' in Bucket.exists(). by dhermes · Pull Request #720 · googleapis/google-cloud-python · GitHub

Using 'fields' in Bucket.exists(). - #720

Merged
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:use-fields-in-bucket-exists
Mar 13, 2015
Merged

Using 'fields' in Bucket.exists().#720
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:use-fields-in-bucket-exists

Conversation

dhermes commented Mar 12, 2015

Copy link
Copy Markdown
Contributor

This decouples Bucket.exists() from Connection.get_bucket().

@tseaver This came up when I was to removing Connection.get_bucket(). I opted to just use connection.api_request instead of dealing with the potential cycle caused by importing storage.api anywhere but __init__.

As it turns out (as I mention in #632), this is a way to reduce the response payload for checking existence (unfortunately, the headers still are overwhelmingly verbose).

This decouples Bucket.exists() from Connection.get_bucket().

Relevant to googleapis#632.
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 12, 2015

Copy link
Copy Markdown

Coverage remained the same at 100.0% when pulling 6186059 on dhermes:use-fields-in-bucket-exists into d9e2156 on GoogleCloudPlatform:master.

dhermes added the api: storage Issues related to the Cloud Storage API. label Mar 12, 2015

tseaver commented Mar 13, 2015

Copy link
Copy Markdown
Contributor

LGTM

dhermes added a commit that referenced this pull request Mar 13, 2015
dhermes merged commit ab642a3 into googleapis:master Mar 13, 2015
dhermes deleted the use-fields-in-bucket-exists branch March 13, 2015 16:28
parthea pushed a commit that referenced this pull request Sep 18, 2025
This prepares the way for using the `query_and_wait` method built-in to the client library when available.

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [ ] Make sure to open an issue as a [bug/issue](https://togithub.com/googleapis/python-bigquery-pandas/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [ ] Ensure the tests and linter pass
- [ ] Code coverage does not decrease (if any source code was changed)
- [ ] Appropriate docs were updated (if necessary)

Towards #710 
🦕
parthea pushed a commit that referenced this pull request Nov 24, 2025
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Nov 25, 2025
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Feb 13, 2026
Removing @googleapis/api-logging, @googleapis/api-logging-partners, and
@googleapis/api-logging-reviewers. Replaced with
@googleapis/yoshi-python in metadata and blunderbuss.

b/476446922
parthea pushed a commit that referenced this pull request Mar 2, 2026
Source-Link: googleapis/synthtool@d2871d9
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:b2dc5f80edcf5d4486c39068c9fa11f7f851d9568eea4dcba130f994ea9b5e97

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Mar 9, 2026
* feat: enable SQLAlchemy 2.0's insertmany feature

- Enable the use_insertmanyvalues flag on the dialect to support
  SQLAlchemy 2.0's insertmany feature, which allows multiple ORM
  objects to be inserted together in bulk, even if the table has
  server-side generated values which must be included in a THEN RETURN
  clause
- Provide an example for using the feature with client-side supplied
  UUIDs and insert_sentinel columns.
- Ensure that the feature is not enables for bit-reversed primary
  keys. In other dialects, an incrementing primary key can be used
  rather than a sentinel column. In Spanner, the bit-reversed integers
  do not meet the ordering requirement to be used as implicit
  sentinels

https://docs.sqlalchemy.org/en/20/core/internals.html#sqlalchemy.engine.default.DefaultDialect.use_insertmanyvalues

https://docs.sqlalchemy.org/en/20/core/connections.html#insert-many-values-behavior-for-insert-statements

https://docs.sqlalchemy.org/en/20/core/connections.html#configuring-sentinel-columns

Fixes: #720

* chore: update tests for multiplexed sessions

* chore: make sample runnable

* fix: exclude commit_timestamp col from THEN RETURN

---------

Co-authored-by: Knut Olav Løite <koloite@gmail.com>
parthea pushed a commit that referenced this pull request Mar 9, 2026
* tests: add retry conf s7 resumable upload test cases

* adjust fixture and chunk size from design discussion
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the Cloud Storage API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL