Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use peewee iterator in loaders #1495

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from
Draft

Conversation

pylipp
Copy link
Contributor

@pylipp pylipp commented Aug 29, 2024

cf. http://docs.peewee-orm.com/en/latest/peewee/querying.html#iterating-over-large-result-sets

To reduce the amount of memory used by peewee when iterating over a
query, use the iterator() method. This method allows you to iterate
without caching each model returned, using much less memory when
iterating over large result sets.

@pylipp pylipp force-pushed the use-peewee-iterator-in-loaders branch from 48300fd to 2861565 Compare January 28, 2025 15:37
Copy link

codecov bot commented Jan 28, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.48%. Comparing base (a933209) to head (2861565).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1495   +/-   ##
=======================================
  Coverage   84.47%   84.48%           
=======================================
  Files         246      246           
  Lines       15462    15464    +2     
  Branches     2248     2248           
=======================================
+ Hits        13062    13064    +2     
  Misses       2359     2359           
  Partials       41       41           
Flag Coverage Δ
backend 99.09% <ø> (+<0.01%) ⬆️
frontend 78.25% <ø> (ø)
sharedComponents 78.45% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

cf. http://docs.peewee-orm.com/en/latest/peewee/querying.html#iterating-over-large-result-sets

> To reduce the amount of memory used by peewee when iterating over a
> query, use the iterator() method. This method allows you to iterate
> without caching each model returned, using much less memory when
> iterating over large result sets.
@pylipp pylipp force-pushed the use-peewee-iterator-in-loaders branch from 2861565 to f92f3cb Compare January 28, 2025 16:40
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.

1 participant