Concurrent Requests

[kelunik]

Currently there's a concurrent request feature, which kind of application really uses that? IMO applications that are able to concurrently process requests should directly use either react/http or amphp/aerys, but do not benefit from php-pm/php-pm.

Should #267 be accepted, can we then remove that?

[marcj]

Yes, we can probably remove that as it causes only issues.

[andig]

IMO applications that are able to concurrently process requests should directly use either react/http or amphp/aerys, but do not benefit from php-pm/php-pm.

I beg to differ. Any application that uses request/ response abstraction and does internally not rely on state should be able to work with concurrent requests. One such example should be the volkszaehler middleware (https://github.com/volkszaehler/volkszaehler.org/blob/master/lib/Router.php) although it's not based on reactPHP.

[kelunik]

@andig Seems like that does blocking dispatching and thus can't support concurrent requests?

[andig]

See #302 for removing concurrent request support.

[andig]

Seems like that does blocking dispatching and thus can't support concurrent requests?

@kelunik that is ... of course true. PR for removal is available.