On 4. 1. 26 22:44, Daniel Sahlberg wrote:
So I added this and linked from SERF-208. I think removing the FIXME's is a blocker (not that it is very difficult), except possibly the TODO.
As I commented in the issue: we can decide that the async resolver is not experimental. It's a new API with limited testing.
The TODO will have to stay for now because I haven't been able to find a way to reliably awake the context. This is because adding or removing file descriptors to the pollset can be completely outside Serf's control, handled by the application. Otherwise we'd just add a wakeup pipe and be done with it.
The thing to watch here (and document) is that any new resolved addresses may not be noticed until the poll timeout has expired, so if an application uses a really long poll delay, this could be a bit of a pain. That's not really recommended for async I/O, of course -- there should ideally be no delay.
-- Brane
