Skip to content

Conversation

@Talakh
Copy link

@Talakh Talakh commented Nov 18, 2025

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)

Fixes #<issue_number_goes_here> ☕️

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

PhongChuong and others added 30 commits December 16, 2024 08:50
…oogleapis#3602)

* feat(bigquery): support IAM conditions in datasets in Java client.

* Fix formatting

* Account for possible null condition field in Acl.

* Add toString() method to Acl.Expr object.

Use service account in integration test instead of hardcoded
personal account.

Change Database API calls to only have one branch, toggling only
the access policy version in a conditional.

* Change Acl.User to be default google credentials in IT test

* fix formatting

* Add Acl.Expr builder. Fix review nits.

* fix formatting
…#3627)

* fix: NPE when reading BigQueryResultSet from empty tables

* Add unittests
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
)

* fix(bigquery): Remove ReadAPI bypass in executeSelect() for fast query requests.

* Enable fast query and read API.

Move readAPI condition check from getExecuteSelectResponse() to
queryRpc(). This allows fast query to be used with ReadAPI.

* Check for null for fast query results.getTotalRows()

* Remove test file.

* Add internal query state to keep track of ReadAPI usage.

A Boolean field is added to keep track of whether or not the high
throughput ReadAPI is used. This is mostly for testing to avoid
another regression in the future.

* Move tests into IT test file

* Fix formatting changes. Again :/

* Add VisibleForTesting Annotation
* Shutdown bqReadClient after high throughput read

* Code formatted to google-java-format
renovate-bot and others added 22 commits September 25, 2025 21:36
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
…3995)

* feat: new queryNoWait method

* Rename method to queryWithTimeout

* rename to queryWithTimeout()

* lint

* lint

* add tests

* Update clirr ignore
…ogleapis#3999)

* chore: grant write access to @googleapis/cloud-java-team-teamsync

* Update CODEOWNERS to simplify ownership rules

Removed specific ownership for handwritten Java libraries.
…storage-bom to v3.18.0 (googleapis#3998)

Co-authored-by: Diego Marquez <diegomarquezp@google.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
# Conflicts:
#	benchmark/pom.xml
#	google-cloud-bigquery-bom/pom.xml
#	google-cloud-bigquery/pom.xml
#	pom.xml
#	samples/native-image-sample/pom.xml
#	samples/snapshot/pom.xml
@Talakh Talakh self-assigned this Nov 18, 2025
@linear
Copy link

linear bot commented Nov 18, 2025

@Talakh Talakh merged commit 6e9b3f9 into master Nov 19, 2025
12 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.