8000
  • #485: Drop Query use of Dataset object by tseaver · Pull Request #498 · googleapis/google-cloud-python · GitHub
    [go: up one dir, main page]

    Skip to content

    #485: Drop Query use of Dataset object#498

    Merged
    tseaver merged 5 commits intogoogleapis:masterfrom
    tseaver:485-remove_dataset_obj_from_query
    Jan 7, 2015
    Merged

    #485: Drop Query use of Dataset object#498
    tseaver merged 5 commits intogoogleapis:masterfrom
    tseaver:485-remove_dataset_obj_from_query

    Conversation

    @tseaver
    Copy link
    Contributor
    @tseaver tseaver commented Jan 7, 2015

    Instead, dataset_id and connection are passed in, or derived from
    the implicit environ.

    Fixes #485.

    Instead, 'dataset_id' and 'connection' are passed in, or derived from
    the implicit environ.
    
    Fixes #485.
    @dhermes
    Copy link
    Contributor
    dhermes commented Jan 7, 2015

    Lint points out you should change Query base class back to object.

    This comment was marked as spam.

    This comment was marked as spam.

    This comment was marked as spam.

    @dhermes
    Copy link
    Contributor
    dhermes commented Jan 7, 2015

    LGTM pending Query subclass fix.

    Feel free to merge, but I'd still love some comments on my comments.

    @coveralls
    Copy link

    Coverage Status

    Coverage remained the same when pulling cec3a39 on tseaver:485-remove_dataset_obj_from_query into 87273a1 on GoogleCloudPlatform:master.

    tseaver added a commit that referenced this pull request Jan 7, 2015
    @tseaver tseaver merged commit 9bafba0 into googleapis:master Jan 7, 2015
    @tseaver tseaver deleted the 485-remove_dataset_obj_from_query branch January 7, 2015 17:41
    @dhermes dhermes added the api: datastore Issues related to the Datastore API. label Dec 31, 2015
    tswast added a commit to tswast/google-cloud-python that referenced this pull request Mar 26, 2019
    Even if the Models API itself is considered GA, I'd like to mark the
    client methods as Beta for a time to give us some more ability to make
    breaking changes until we're sure of the public interface we want to
    expose.
    atulep pushed a commit that referenced this pull request Apr 6, 2023
    * chore(python): upgrade gcp-releasetool in .kokoro [autoapprove]
    
    Source-Link: googleapis/synthtool@5f2a608
    Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:8555f0e37e6261408f792bfd6635102d2da5ad73f8f09bcb24f25e6afb5fac97
    
    * trigger ci
    
    ---------
    
    Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
    Co-authored-by: Anthonios Partheniou <partheniou@google.com>
    atulep pushed a commit that referenced this pull request Apr 18, 2023
    * chore(python): upgrade gcp-releasetool in .kokoro [autoapprove]
    
    Source-Link: googleapis/synthtool@5f2a608
    Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:8555f0e37e6261408f792bfd6635102d2da5ad73f8f09bcb24f25e6afb5fac97
    
    * trigger ci
    
    ---------
    
    Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
    Co-authored-by: Anthonios Partheniou <partheniou@google.com>
    parthea pushed a commit that referenced this pull request Aug 15, 2023
    Source-Link: https://togithub.com/googleapis/synthtool/commit/26c7505b2f76981ec1707b851e1595c8c06e90fc
    Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:f946c75373c2b0040e8e318c5e85d0cf46bc6e61d0a01f3ef94d8de974ac6790
    vchudnov-g pushed a commit that referenced this pull request Sep 20, 2023
    Source-Link: googleapis/synthtool@993985f
    Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:1894490910e891a385484514b22eb5133578897eb5b3c380e6d8ad475c6647cd
    parthea pushed a commit that referenced this pull request Sep 22, 2023
    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: Update gapic-generator-python to v1.11.2
    
    PiperOrigin-RevId: 546510849
    
    Source-Link: googleapis/googleapis@736073a
    
    Source-Link: googleapis/googleapis-gen@deb64e8
    Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiZGViNjRlOGVjMTlkMTQxZTMxMDg5ZmU5MzJiM2E5OTdhZDU0MWM0ZCJ9
    
    * 🦉 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>
    parthea added a commit that referenced this pull request Oct 21, 2023
    * chore: update to gapic-generator-python 1.5.0
    
    feat: add support for `google.cloud.<api>.__version__`
    PiperOrigin-RevId: 484665853
    
    Source-Link: googleapis/googleapis@8eb249a
    
    Source-Link: googleapis/googleapis-gen@c8aa327
    Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiYzhhYTMyN2I1ZjQ3ODg2NWZjM2ZkOTFlM2MyNzY4ZTU0ZTI2YWQ0NCJ9
    
    * 🦉 Updates from OwlBot post-processor
    
    See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
    
    * chore: Update to gapic-generator-python 1.6.0
    
    feat(python): Add typing to proto.Message based class attributes
    
    feat(python): Snippetgen handling of repeated enum field
    
    PiperOrigin-RevId: 487326846
    
    Source-Link: googleapis/googleapis@da380c7
    
    Source-Link: googleapis/googleapis-gen@61ef576
    Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNjFlZjU3NjJlZTY3MzFhMGNiYmZlYTIyZmQwZWVjZWU1MWFiMWM4ZSJ9
    
    * 🦉 Updates from OwlBot post-processor
    
    See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
    
    * feat: new APIs added to reflect updates to the filestore service
    
    - Add ENTERPRISE Tier
    - Add snapshot APIs: RevertInstance, ListSnapshots, CreateSnapshot, DeleteSnapshot, UpdateSnapshot
    - Add multi-share APIs: ListShares, GetShare, CreateShare, DeleteShare, UpdateShare
    - Add ConnectMode to NetworkConfig (for Private Service Access support)
    - New status codes (SUSPENDED/SUSPENDING, REVERTING/RESUMING)
    - Add SuspensionReason (for KMS related suspension)
    - Add new fields to Instance information: max_capacity_gb, capacity_step_size_gb, max_share_count, capacity_gb, multi_share_enabled
    
    PiperOrigin-RevId: 487492758
    
    Source-Link: googleapis/googleapis@5be5981
    
    Source-Link: googleapis/googleapis-gen@ab0e217
    Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiYWIwZTIxN2Y1NjBjYzJjMWFmYzExNDQxYzJlYWI2YjY5NTBlZmQyYiJ9
    
    * 🦉 Updates from OwlBot post-processor
    
    See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
    
    * chore: Update gapic-generator-python to v1.6.1
    
    PiperOrigin-RevId: 488036204
    
    Source-Link: googleapis/googleapis@08f275f
    
    Source-Link: googleapis/googleapis-gen@555c094
    Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiNTU1YzA5NDVlNjA2NDllMzg3MzlhZTY0YmM0NTcxOWNkZjcyMTc4ZiJ9
    
    * 🦉 Updates from OwlBot post-processor
    
    See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
    
    * chore: use templated setup.py and owlbot.py
    
    * remove v1p1beta1 docs
    
    * set coverage to 99
    
    * remove commented code
    
    * fix(deps): Require google-api-core >=1.34.0, >=2.11.0
    
    fix: Drop usage of pkg_resources
    
    fix: Fix timeout default values
    
    docs(samples): Snippetgen should call await on the operation coroutine before calling result
    
    PiperOrigin-RevId: 493260409
    
    Source-Link: googleapis/googleapis@fea4387
    
    Source-Link: googleapis/googleapis-gen@387b734
    Copy-Tag: eyJwIjoiLmdpdGh1Yi8uT3dsQm90LnlhbWwiLCJoIjoiMzg3YjczNDRjNzUyOWVlNDRiZTg0ZTYxM2IxOWE4MjA1MDhjNjEyYiJ9
    
    * 🦉 Updates from OwlBot post-processor
    
    See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md
    
    * add gapic_version.py
    
    Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
    Co-authored-by: Anthonios Partheniou <partheniou@google.com>
    parthea pushed a commit that referenced this pull request Oct 21, 2023
    Source-Link: googleapis/synthtool@eaef28e
    Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:f8ca7655fa8a449cadcabcbce4054f593dcbae7aeeab34aa3fcc8b5cf7a93c9e
    
    Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
    parthea added a commit that referenced this pull request Oct 22, 2023
    * chore(python): upgrade gcp-releasetool in .kokoro [autoapprove]
    
    Source-Link: googleapis/synthtool@5f2a608
    Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:8555f0e37e6261408f792bfd6635102d2da5ad73f8f09bcb24f25e6afb5fac97
    
    * trigger ci
    
    ---------
    
    Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
    Co-authored-by: Anthonios Partheniou <partheniou@google.com>
    parthea added a commit that referenced this pull request Sep 18, 2025
    * chore(deps): update all dependencies
    
    * revert change
    
    Co-authored-by: Anthonios Partheniou <partheniou@google.com>
    parthea pushed a commit that referenced this pull request Nov 22, 2025
    parthea added a commit that referenced this pull request Nov 24, 2025
    * chore: update templated files
    
    * 🦉 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>
    parthea added a commit that referenced this pull request Nov 24, 2025
    * fix: add actionable errors for GCE long running operations
    
    * add unit test
    
    * mypy
    
    * add notes that the workaround should be removed once proposal A from b/284179390 is implemented
    
    * fix typo
    
    * fix coverage
    parthea pushed a commit that referenced this pull request Nov 24, 2025
    Source-Link: googleapis/synthtool@f58d313
    Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:7a40313731a7cb1454eef6b33d3446ebb121836738dc3ab3d2d3ded5268c35b6
    
    Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
    parthea added a commit that referenced this pull request Nov 24, 2025
    * 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 python 3.12 to setup.py, constraints and required checks
    
    ---------
    
    Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
    Co-authored-by: Anthonios Partheniou <partheniou@google.com>
    Co-authored-by: Daniel Sanche <sanche@google.com>
    parthea pushed a commit that referenced this pull request Nov 26, 2025
    * fix: fix the usage so the tests can pass in g3
    
    * use frozenset
    parthea pushed a commit that referenced this pull request Mar 2, 2026
    parthea pushed a commit that referenced this pull request Mar 2, 2026
    This PR replaces the old yoshi-python team with cloud-sdk-python-team
    and cloud-storage-dpe with gcs-team.
    
    b/478003109
    parthea pushed a commit that referenced this pull request Mar 9, 2026
    * fix: support THEN RETURN for insert, update, delete
    
    Support THEN RETURN clauses for INSERT, UPDATE, and DELETE statements.
    
    Fixes #498
    
    * test: override insert auto-generated pk test
    parthea pushed a commit that referenced this pull request Mar 9, 2026
    Test hygeine: avoid importing module-under-test at module scope
    
    Closes #498.
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Labels

    api: datastore Issues related to the Datastore API.

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    Remove Query dependency on Dataset and make dataset_id required in Query

    3 participants

    0