Skip to content
This repository was archived by the owner on Apr 1, 2026. It is now read-only.

feat: row_exists and read_row - #778

Merged
daniel-sanche merged 373 commits into
googleapis:v3from
daniel-sanche:read_rows_helpers
Jun 16, 2023
Merged

feat: row_exists and read_row#778
daniel-sanche merged 373 commits into
googleapis:v3from
daniel-sanche:read_rows_helpers

Conversation

@daniel-sanche

@daniel-sanchedaniel-sanche commented May 18, 2023

Copy link
Copy Markdown
Contributor

This PR adds two useful helper functions:

  • row_exists: returns a bool indicating whether the row was found on the backend, without overhead of actually fetching its data
  • read_row: returns an individual row from the backend, or None if it doesn't exist

@product-auto-labelproduct-auto-labelBot added size: xl Pull request size is extra large. api: bigtable Issues related to the googleapis/python-bigtable API. labels May 18, 2023
@product-auto-labelproduct-auto-labelBot added size: m Pull request size is medium. and removed size: xl Pull request size is extra large. labels May 24, 2023
@daniel-sanchedaniel-sanche changed the title [DRAFT] feat: read_rows helpersfeat: read_rows helpersMay 30, 2023
@daniel-sanche
daniel-sanche marked this pull request as ready for review May 30, 2023 16:50
@daniel-sanche
daniel-sanche requested review from a teamMay 30, 2023 16:50
@daniel-sanchedaniel-sanche added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 7, 2023
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 7, 2023
@daniel-sanchedaniel-sanche changed the title feat: read_rows helpersfeat: row_exists and read_rowJun 8, 2023
Comment threadgoogle/cloud/bigtable/client.py
Comment threadgoogle/cloud/bigtable/client.py
@product-auto-labelproduct-auto-labelBot added size: l Pull request size is large. and removed size: m Pull request size is medium. labels Jun 14, 2023

@mutianfmutianf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please add the retry wrapper for read_row

@daniel-sanche
daniel-sanche merged commit 5d65703 into googleapis:v3Jun 16, 2023
daniel-sanche added a commit that referenced this pull request Feb 5, 2024
* feat: add new v3.0.0 API skeleton (#745)
* feat: improve rows filters (#751)
* feat: read rows query model class (#752)
* feat: implement row and cell model classes (#753)
* feat: add pooled grpc transport (#748)
* feat: implement read_rows (#762)
* feat: implement mutate rows (#769)
* feat: literal value filter (#767)
* feat: row_exists and read_row (#778)
* feat: read_modify_write and check_and_mutate_row (#780)
* feat: sharded read rows (#766)
* feat: ping and warm with metadata (#810)
* feat: mutate rows batching (#770)
* chore: restructure module paths (#816)
* feat: improve timeout structure (#819)
* fix: api errors apply to all bulk mutations
* chore: reduce public api surface (#820)
* feat: improve error group tracebacks on < py11 (#825)
* feat: optimize read_rows (#852)
* chore: add user agent suffix (#842)
* feat: optimize retries (#854)
* feat: add test proxy (#836)
* chore(tests): add conformance tests to CI for v3 (#870)
* chore(tests): turn off fast fail for conformance tets (#882)
* feat: add TABLE_DEFAULTS enum for table method arguments (#880)
* fix: pass None for retry in gapic calls (#881)
* feat: replace internal dictionaries with protos in gapic calls (#875)
* chore: optimize gapic calls (#863)
* feat: expose retryable error codes to users (#879)
* chore: update api_core submodule (#897)
* chore: merge main into experimental_v3 (#900)
* chore: pin conformance tests to v0.0.2 (#903)
* fix: bulk mutation eventual success (#909)
---------
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigtableIssues related to the googleapis/python-bigtable API.size: lPull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@daniel-sanche@mutianf@yoshi-kokoro