search

sociomantic-tsunami / dlsproto

Distributed Log Store protocol definition, client, fake node, and tests
Boost Software License 1.0
3 stars 18 forks source link

Unregister the timer and remove it from the SelectedKeysHandler list #104

Closed nemanja-boric-sociomantic closed 6 years ago

nemanja-boric-sociomantic commented 6 years ago

This unregisters the timer from the epoll, but it also removes it from the set of ready select clients to handle, in it's ready to fire, but the request was already out of the scope.

nemanja-boric-sociomantic commented 6 years ago

I failed to reproduce the original issue (#102), but this fix is reasonable thing to do, even if the cause of #102 is something else (which I doubt).

gavin-norman-sociomantic commented 6 years ago

I failed to reproduce the original issue (#102), but this fix is reasonable thing to do, even if the cause of #102 is something else (which I doubt).

Ok cool, let's deploy this and see if the problem disappears.