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

feat: skip large rows - #2482

Merged
gcf-merge-on-green[bot] merged 31 commits into
googleapis:mainfrom
sarthakbhutani:large-row-skip
Feb 21, 2025
Merged

gcf-merge-on-green[bot] merged 31 commits into
googleapis:mainfrom
sarthakbhutani:large-row-skip

Conversation

@sarthakbhutani

Copy link
Copy Markdown
Contributor

Tasks remaining -

  • make changes in the read request API which skips for large rows/internally calls readLargeRowsCallable()
  • expose large rows rowkeys in sidechannel/dlq/some other method - which can be exposed to client

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 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)
  • Rollback plan is reviewed and LGTMed
  • All new data plane features have a completed end to end testing plan

Fixes #<issue_number_goes_here> ☕️

If you write sample code, please follow the samples format.

Loading
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: bigtable Issues related to the googleapis/java-bigtable API. size: xl Pull request size is extra large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants