Skip to content
This repository has been archived by the owner on Apr 2, 2019. It is now read-only.

Resolving maximum call stack size exceeded issue #28

Closed

Conversation

MGardner02
Copy link

This is a fix for #26 which seems to occur when too much data is queued up at once and then return using the recursive call. We saw the error coming up when reading files with millions of lines and when pulling thousands of results from a DB. I updated the code to follow the same logic as before, but use a loop instead to avoid recursion issues.

I read through another thread and saw this was discussed as a potential solution, the update passed all unit testing and worked well when I tested against a few additional stream examples. Let me know your thoughts.

@dominictarr
Copy link
Owner

sorry, I no longer have access to merge and publish changes to this module. You'll need to publish this as map-stream2 (or whatever you choose) and maintain it yourself)

@dominictarr dominictarr closed this Apr 2, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants