This repository has been archived by the owner on May 3, 2022. It is now read-only.

Node: Implement back pressure in v2/handler #262

Open

Raynos opened this issue Mar 27, 2015 · 1 comment

Labels

node

Contributor

Raynos commented Mar 27, 2015

We currently do not implement back pressure.

we just push() frames forever.

The text was updated successfully, but these errors were encountered:

Raynos added the node label

Contributor Author

Raynos commented Jun 16, 2015

We still do not have backpressure.

The code is different though;

v2/handler.js implements handleFrame()
connection.js calls it from handleReadFrame()
the bufrw/stream/read_machine drives handleReadFrame()
read_machine does not implement pause() and v2/handler.js does not return any kind of "I'm full" message.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.