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

Closing a Finagle service leaves live, non-daemon threads #5

Closed
stuhood opened this issue Feb 1, 2011 · 9 comments
Closed

Closing a Finagle service leaves live, non-daemon threads #5

stuhood opened this issue Feb 1, 2011 · 9 comments

Comments

@stuhood
Copy link
Contributor

stuhood commented Feb 1, 2011

Starting a Finagle client using ClientBuilder (and the ServiceToClient Thrift generation in this case), using it, and then closing it down will cause the process to stay alive, probably because of non-daemon threads opened by Finagle. Attaching a stack from after calling close on the Service underlying the ServiceToClient object.

Thanks!

@stuhood
Copy link
Contributor Author

stuhood commented Feb 1, 2011

@mariusae
Copy link
Contributor

mariusae commented Feb 2, 2011

this should be fixed in 1.1.10 — could you verify? thanks, marius.

@stuhood
Copy link
Contributor Author

stuhood commented Feb 2, 2011

I think the 1.1.10 release is causing other problems. With or without our custom Protocol shim, we're getting: https://gist.github.com/808398 ... on the first request.

@mariusae
Copy link
Contributor

mariusae commented Feb 3, 2011

thanks for the stack trace. i was able to reproduce & fix.

80a0794

this is now included in the 1.1.11 release.

@stuhood
Copy link
Contributor Author

stuhood commented Feb 3, 2011

The problem from 1.1.10 is fixed, but I'm still seeing the original issue with 1.1.11: I've updated this gist https://gist.github.com/806609 with the new stack trace I see after exit. In particular, there is one thread in some Netty code for "HashedWheelTimer" that looks suspicious.

@stuhood
Copy link
Contributor Author

stuhood commented Feb 4, 2011

Sorry, don't know how this got closed: as the last comment indicates, it is still a problem.

@mariusae
Copy link
Contributor

mariusae commented Feb 5, 2011

ok, cool. 1.1.11 squashed the NIO worker threads, but i still need to refcount the timer thread.

i'll fix this on monday.

@mariusae
Copy link
Contributor

mariusae commented Feb 7, 2011

this should be resolved in 1.1.15.

let me know.

@stuhood
Copy link
Contributor Author

stuhood commented Feb 8, 2011

Indeed it is! Thank you very much Marius.

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants