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

io.aviso.taxi-toolkit.utils/retry consumes the stack #12

Open
hlship opened this issue Jun 18, 2015 · 1 comment
Open

io.aviso.taxi-toolkit.utils/retry consumes the stack #12

hlship opened this issue Jun 18, 2015 · 1 comment

Comments

@hlship
Copy link
Collaborator

hlship commented Jun 18, 2015

The retry function uses stack recursion, rather than the recur special form, and so each retry adds a stack frame every 100 ms.

@kamituel
Copy link
Contributor

kamituel commented Jul 7, 2015

Is that an issue in the real world? I mean, (retry) is only repeating an action for one second, which means that we'd recur up to max 10 times.

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

No branches or pull requests

2 participants