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

feat: clean cached rows and responses after conversion#1393

Merged
alvarowolfx merged 5 commits intogoogleapis:mainfrom
alvarowolfx:feat-reduce-memory-rows-resp
Jul 31, 2024
Merged

feat: clean cached rows and responses after conversion#1393
alvarowolfx merged 5 commits intogoogleapis:mainfrom
alvarowolfx:feat-reduce-memory-rows-resp

Conversation

@alvarowolfx
Copy link
Copy Markdown
Contributor

Reduce memory footprint when fetching data.

Tests running query: 'SELECT repository_url as url, repository_owner as owner, repository_forks as forks FROM bigquery-public-data.samples.github_timeline where repository_url is not null LIMIT 1000000'

Before change: ~70mb of heap memory usage
After change: ~35mb of heap memory usage

Towards #1392

@alvarowolfx alvarowolfx requested review from a team and shollyman July 11, 2024 14:56
@product-auto-label product-auto-label Bot added size: s Pull request size is small. api: bigquery Issues related to the googleapis/nodejs-bigquery API. labels Jul 11, 2024
@alvarowolfx alvarowolfx added the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 11, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 11, 2024
Copy link
Copy Markdown
Contributor

@shollyman shollyman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

heaps of goodness!

@shollyman shollyman added the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 29, 2024
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jul 29, 2024
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/nodejs-bigquery API. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants