Skip to content

fix: include amended user agent in read client#17592

Merged
shuoweil merged 8 commits into
mainfrom
b526614511-tswast-pandas-user-agent
Jul 1, 2026
Merged

fix: include amended user agent in read client#17592
shuoweil merged 8 commits into
mainfrom
b526614511-tswast-pandas-user-agent

Conversation

@tswast

@tswast tswast commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Towards internal issue b/526614511 🦕

@tswast tswast requested review from a team as code owners June 29, 2026 21:33
@tswast tswast requested review from TrevorBergeron and removed request for a team June 29, 2026 21:33

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request reorganizes imports across several files and updates the _ensure_bqstorage_client method in client.py to append pandas-gbq version information to the client_info user agent when initializing the BigQueryReadClient. It also adds comprehensive unit tests to verify this behavior under different scenarios. The review feedback suggests adding a # type: ignore[assignment] comment to the fallback assignment pandas_gbq = None to prevent potential mypy type-checking errors.

Comment thread packages/google-cloud-bigquery/google/cloud/bigquery/client.py Outdated
tswast and others added 2 commits June 29, 2026 21:36
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Comment thread packages/google-cloud-bigquery/google/cloud/bigquery/client.py
@parthea

parthea commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Switching to draft until presubmits are green

@parthea parthea marked this pull request as draft June 30, 2026 13:32
Comment thread packages/google-cloud-bigquery/tests/unit/test_legacy_types.py Outdated
@tswast tswast marked this pull request as ready for review July 1, 2026 13:56
@tswast tswast requested review from shuoweil and removed request for TrevorBergeron July 1, 2026 13:56
@shuoweil shuoweil merged commit c43caee into main Jul 1, 2026
30 checks passed
@shuoweil shuoweil deleted the b526614511-tswast-pandas-user-agent branch July 1, 2026 17:30
@release-please release-please Bot mentioned this pull request Jul 1, 2026
g-husam pushed a commit that referenced this pull request Jul 1, 2026
Towards internal issue b/526614511 🦕

---------

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
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.

3 participants