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

Memory leak #5

Open
virtualfunction opened this issue Feb 9, 2015 · 2 comments
Open

Memory leak #5

virtualfunction opened this issue Feb 9, 2015 · 2 comments

Comments

@virtualfunction
Copy link

  get '/', provides: 'text/event-stream' do
    sse_stream &:close
  end

Just doing this and running this about 10 times shows an increase in mem usage.

httperf --server=localhost --port=3000 --uri=/ --num-conns=500 --rate=500

I've used EM.add_periodic_timer(5) { CG.start } and found it doesn't get collected.

@virtualfunction
Copy link
Author

Think EM is to blame here, see: eventmachine/eventmachine#566

@radiospiel
Copy link
Owner

Jason, thank you for looking into that. I am not able to look into the issue right now, but I would happily accept patches if you find sinatra-sse to blame (or just a workaround.)

@radiospiel radiospiel added ready and removed ready labels Feb 23, 2015
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