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

chore: optimize gapic calls - #863

Merged
daniel-sanche merged 5 commits into
experimental_v3from
optimize-gapic
Dec 1, 2023
Merged

chore: optimize gapic calls#863
daniel-sanche merged 5 commits into
experimental_v3from
optimize-gapic

Conversation

@daniel-sanche

@daniel-sanchedaniel-sanche commented Sep 7, 2023

Copy link
Copy Markdown
Contributor

This PR pulls in changes from googleapis/gapic-generator-python#1755 and googleapis/python-api-core#527 to improve rpc performance.

I measured this change as giving a 6% throughput boost to the point read benchmark in a prod environment

Fixes#863

@product-auto-labelproduct-auto-labelBot added size: m Pull request size is medium. api: bigtable Issues related to the googleapis/python-bigtable API. labels Sep 7, 2023
@daniel-sanchedaniel-sanche added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 12, 2023
@yoshi-kokoroyoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 12, 2023
@daniel-sanche
daniel-sanche marked this pull request as ready for review October 25, 2023 17:50
@daniel-sanche
daniel-sanche requested review from a teamOctober 25, 2023 17:50
Comment threadgoogle/cloud/bigtable_v2/services/bigtable/async_client.py Outdated
@daniel-sanche
daniel-sanche merged commit b191451 into experimental_v3Dec 1, 2023
@daniel-sanche
daniel-sanche deleted the optimize-gapic branch December 1, 2023 21:10
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: mPull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@daniel-sanche@igorbernstein2@yoshi-kokoro