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

fix: remove deepcopy while setting properties for _QueryResults - #2280

Merged
tswast merged 1 commit into
googleapis:mainfrom
Linchin:deepcopy
Sep 6, 2025
Merged

tswast merged 1 commit into
googleapis:mainfrom
Linchin:deepcopy

Conversation

@Linchin

@Linchin Linchin commented Sep 5, 2025 •

Copy link
Copy Markdown
Contributor

See internal issue 442629997

This deepcopy() is causing memory spike when people call Client.query_and_wait().

Fixes #2279 🦕

@Linchin
Linchin requested review from a team and leahecole September 5, 2025 21:01
@product-auto-label product-auto-label Bot added the size: xs Pull request size is extra small. label Sep 5, 2025
@product-auto-label product-auto-label Bot added the api: bigquery Issues related to the googleapis/python-bigquery API. label Sep 5, 2025

@tswast tswast 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!

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

Labels

api: bigquery Issues related to the googleapis/python-bigquery API. size: xs Pull request size is extra small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Memory spike caused by deepcopy in _QueryResult

2 participants