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

Lazily perform searches by yielding results instead of collecting into an array #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

matthewryanscott
Copy link

@matthewryanscott matthewryanscott commented May 21, 2016

With large product and order sets, out-of-memory situations can occur on memory-constrained systems such as Heroku dynos.

This lets you lazily load search results; additional results are only fetched when the previous batch is exhausted.

@matthewryanscott
Copy link
Author

Actually meant to open this in our fork, but I'll leave this open in case your team finds this useful!

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