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

maximum recursion depth exceeded under heavy use of a Limiter #3570

Closed
richvdh opened this issue Jul 20, 2018 · 1 comment
Closed

maximum recursion depth exceeded under heavy use of a Limiter #3570

richvdh opened this issue Jul 20, 2018 · 1 comment

Comments

@richvdh
Copy link
Member

richvdh commented Jul 20, 2018

I fixed this a while ago for the Linearizer in #2532, but apparently failed to do the same thing for the Limiter.

When we have lots of things waiting for a Limiter, and then a bunch of them complete synchronously, we can end up hitting the stack limits and breaking everything.
richvdh added a commit that referenced this issue Jul 20, 2018
@richvdh
Add a sleep to the Limiter to fix stack overflows.
d7275ee 
Fixes #3570
@richvdh richvdh mentioned this issue Jul 20, 2018
Merged
@richvdh
Copy link
Member Author

richvdh commented Jul 23, 2018

this will have been fixed by #3571

@richvdh richvdh closed this as completed Jul 23, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@richvdh