Remy Maucherat wrote:
Remy Maucherat wrote:
Hi,

To more conveniently support some use cases, a new notify method could be added to the event. This would do an action callback and call a new method in the Poller class would add the socket to a structure like the socket add list. The comet poller thread will then check if there are sockets in that list, and call an event (as for the event type, it could be a regular timeout - it's very similar - or a new type which will behave the same as timeout - which would be significantly harder since modifications will have to be made to accommodate the new event type).

I know how to make this fairly easily with the APR connector, and I'm not so sure about the NIO one (most likely it's very similar, however). The main thing which is annoying to implement is if the fact that there will not be any other concurrent event occurring has to be enforced and there are multiple pollers (I think the socket has to be removed "just in case" from each poller, which is quite annoying).

Along with it, I think a semi dynamic option should be added to not use read polling (which does funky stuff in certain cases, since when using pipelining, there's data available but the data may actually not belong to this request), but still check timeout. I think per connection timeout should also be supported by APR. Having these in addition to n.tify() would allow all possible usage scenarios without running into funny corner cases.

Oh by the way, before we start doing all this, I'd really like to push out another "stable" 6.0. There's been tons of bug fixes, and before we put everything on the chopping block again, lets push those out.

Filip


Rémy

---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]





---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to