KEMBAR78
fix(bigquery): cache total rows count by alvarowolfx · Pull Request #12230 · googleapis/google-cloud-go · GitHub
Skip to content

Conversation

alvarowolfx
Copy link
Contributor

Total rows on fast query path was using the amount of rows returned on the first page, which was incorrect for result sets with multiple pages.

Supersedes #11874
Fixes #11873

@alvarowolfx alvarowolfx requested a review from shollyman May 9, 2025 15:08
@alvarowolfx alvarowolfx requested review from a team as code owners May 9, 2025 15:08
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the BigQuery API. label May 9, 2025
@alvarowolfx alvarowolfx added the automerge Merge the pull request once unit tests and other checks pass. label May 9, 2025
@gcf-merge-on-green gcf-merge-on-green bot merged commit 202dce0 into main May 9, 2025
9 checks passed
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label May 9, 2025
@gcf-merge-on-green gcf-merge-on-green bot deleted the bq-fix-cached-rows branch May 9, 2025 17:14
2FaceS-bit pushed a commit to 2FaceS-bit/google-cloud-go that referenced this pull request May 12, 2025
Total rows on fast query path was using the amount of rows returned on the first page, which was incorrect for result sets with multiple pages.

Supersedes googleapis#11874
Fixes googleapis#11873
gcf-merge-on-green bot pushed a commit that referenced this pull request May 12, 2025
🤖 I have created a release *beep* *boop*
---


## [1.68.0](https://togithub.com/googleapis/google-cloud-go/compare/bigquery/v1.67.0...bigquery/v1.68.0) (2025-05-12)


### Features

* **bigquery/analyticshub:** Support new feature Sharing Cloud Pubsub Streams via AH (GA) and Subscriber Email logging feature ([#11908](https://togithub.com/googleapis/google-cloud-go/issues/11908)) ([a21d596](https://togithub.com/googleapis/google-cloud-go/commit/a21d5965fa3f4322da9563425350ba1079279d5a))
* **bigquery/storage:** Increased the number of partitions can be written in a single request ([43bc515](https://togithub.com/googleapis/google-cloud-go/commit/43bc51591e4ffe7efc76449bb00e3747cda2c944))
* **bigquery:** Add performance insights ([#12101](https://togithub.com/googleapis/google-cloud-go/issues/12101)) ([aef68ab](https://togithub.com/googleapis/google-cloud-go/commit/aef68abaa336e0ecd1f488ef6cb3d6b0e8930835))
* **bigquery:** Add some missing fields to BQ stats  ([#12212](https://togithub.com/googleapis/google-cloud-go/issues/12212)) ([77b08e8](https://togithub.com/googleapis/google-cloud-go/commit/77b08e8e72ece0d56ff8f86dcbfe44b944ab083f))
* **bigquery:** Add WriteTruncateData write disposition ([#12013](https://togithub.com/googleapis/google-cloud-go/issues/12013)) ([b1126a3](https://togithub.com/googleapis/google-cloud-go/commit/b1126a3580a0c81c1d7df7cf138d17c748adefbc))
* **bigquery:** New client(s) ([#12228](https://togithub.com/googleapis/google-cloud-go/issues/12228)) ([f229bd9](https://togithub.com/googleapis/google-cloud-go/commit/f229bd9b90830d96781d3f9059b64dbfece1690b))
* **bigquery:** Support managed iceberg tables ([#11931](https://togithub.com/googleapis/google-cloud-go/issues/11931)) ([35e0774](https://togithub.com/googleapis/google-cloud-go/commit/35e0774bf17166dbaa88eba286f40ad91d9aa68a))
* **bigquery:** Support per-job reservation assignment ([#12078](https://togithub.com/googleapis/google-cloud-go/issues/12078)) ([c9cebcc](https://togithub.com/googleapis/google-cloud-go/commit/c9cebcceebc5fb5eecacf99e18652e0c2a53cc6c))


### Bug Fixes

* **bigquery:** Cache total rows count ([#12230](https://togithub.com/googleapis/google-cloud-go/issues/12230)) ([202dce0](https://togithub.com/googleapis/google-cloud-go/commit/202dce02888c5d1d2821732145d5780e5c07ba05)), refs [#11874](https://togithub.com/googleapis/google-cloud-go/issues/11874) [#11873](https://togithub.com/googleapis/google-cloud-go/issues/11873)
* **bigquery:** Parse timestamps with timezone info ([#11950](https://togithub.com/googleapis/google-cloud-go/issues/11950)) ([530d522](https://togithub.com/googleapis/google-cloud-go/commit/530d522a1f8622e51310680cce31ff1dae007f81))
* **bigquery:** Update google.golang.org/api to 0.229.0 ([3319672](https://togithub.com/googleapis/google-cloud-go/commit/3319672f3dba84a7150772ccb5433e02dab7e201))
* **bigquery:** Upgrade gRPC service registration func ([7c01015](https://togithub.com/googleapis/google-cloud-go/commit/7c01015f2aafb5eeb0237accced76b059bc7635d))


### Documentation

* **bigquery/storage:** Updated the number of partitions (from 100 to 900) can be inserted, updated and deleted in a single request ([43bc515](https://togithub.com/googleapis/google-cloud-go/commit/43bc51591e4ffe7efc76449bb00e3747cda2c944))

---
This PR was generated with [Release Please](https://togithub.com/googleapis/release-please). See [documentation](https://togithub.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigquery Issues related to the BigQuery API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bigquery: total rows has incorrect value

2 participants