Skip to content

bigtable: Require google-cloud-bigtable>=2.42.0 and test Python BigtableIO write error surfacing - #39820

Merged
damccorm merged 1 commit into
apache:masterfrom
mutianf:bigtable-python-batcher-error-handling
Aug 20, 2026
Merged

bigtable: Require google-cloud-bigtable>=2.42.0 and test Python BigtableIO write error surfacing#39820
damccorm merged 1 commit into
apache:masterfrom
mutianf:bigtable-python-batcher-error-handling

Conversation

@mutianf

@mutianfmutianf commented Aug 19, 2026

Copy link
Copy Markdown
Contributor
  • Bump the minimum google-cloud-bigtable to >=2.42.0,<3. 2.42.0 improves MutationsBatcher error handling so that errors raised during async flushes are surfaced instead of swallowed.
  • Add _BigTableWriteFn write tests asserting that batch errors are surfaced (and thus fail the bundle, so the runner retries) from:
    • the async flush path,
    • the buffered-row flush at finish_bundle()/close(),
    • close() itself.
  • Add a user agent in the python client

Previously a batch-level failure during an asynchronous flush could be dropped without failing the bundle. Requiring 2.42.0 and adding these regression tests guards the error-surfacing behavior end-to-end through Beam.

Fix#39755

google-cloud-bigtable 2.42.0 improves MutationsBatcher error handling so
that errors raised during async flushes are surfaced instead of being
swallowed. Bump the minimum version and add BigtableIO write tests that
assert batch errors are surfaced (and fail the bundle) from the async
flush path, the buffered-row flush at finish_bundle, and close() itself.
@mutianf
mutianfforce-pushed the bigtable-python-batcher-error-handling branch from 151f342 to c50d361CompareAugust 19, 2026 16:27
@github-actions

Copy link
Copy Markdown
Contributor

Assigning reviewers:

R: @shunping for label python.

Note: If you would like to opt out of this review, comment assign to next reviewer.

Available commands:

  • stop reviewer notifications - opt out of the automated review tooling
  • remind me after tests pass - tag the comment author after tests pass
  • waiting on author - shift the attention set back to the author (any comment or push by the author will return the attention set to the reviewers)

The PR bot will only process comments in the main thread (not review comments).

@damccormdamccorm 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.

Thanks!

@damccorm
damccorm merged commit 93f3e05 into apache:masterAug 20, 2026
110 of 112 checks passed
damccorm added a commit that referenced this pull request Aug 20, 2026
…39820) (#39831)
google-cloud-bigtable 2.42.0 improves MutationsBatcher error handling so
that errors raised during async flushes are surfaced instead of being
swallowed. Bump the minimum version and add BigtableIO write tests that
assert batch errors are surfaced (and fail the bundle) from the async
flush path, the buffered-row flush at finish_bundle, and close() itself.
Co-authored-by: Mattie Fu <mattiefu@google.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Bigtable Python SDK upgrade

2 participants

@mutianf@damccorm