diff options
author | Wladimir J. van der Laan <laanwj@gmail.com> | 2015-11-11 17:34:10 +0100 |
---|---|---|
committer | Wladimir J. van der Laan <laanwj@gmail.com> | 2015-11-13 11:10:48 +0100 |
commit | a264c32e3321ae909ca59cb8ce8bf5d812dbc4e1 (patch) | |
tree | 2f934da3736223e127356a496b0bfb6fa4563fca /COPYING | |
parent | 3ac70609345a249a74fad61b70f93e8a19245011 (diff) |
http: speed up shutdown
This continues/fixes #6719.
`event_base_loopbreak` was not doing what I expected it to, at least in
libevent 2.0.21.
What I expected was that it sets a timeout, given that no other pending
events it would exit in N seconds. However, what it does was delay the
event loop exit with 10 seconds, even if nothing is pending.
Solve it in a different way: give the event loop thread time to exit
out of itself, and if it doesn't, send loopbreak.
This speeds up the RPC tests a lot, each exit incurred a 10 second
overhead, with this change there should be no shutdown overhead in the
common case and up to two seconds if the event loop is blocking.
As a bonus this breaks dependency on boost::thread_group, as the HTTP
server minds its own offspring.
Diffstat (limited to 'COPYING')
0 files changed, 0 insertions, 0 deletions