-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bring in a faster Transform stream #147
Comments
I don't think we should have another stream implementation, this will be confusing and need to maintain multiple implementations for complex topic |
There is room to further optimize |
Also rather than adding another stream api, I think we should add support for sync functions in pipeline, i.e.
|
I would love this, whoever the world depends on
@ronag that would have almost the same effect. Maybe we could internalize SyncThrough or analog and use ti to support pipeline |
@ronag I remember u had some pr to remove the double buffering but stopped, do you remember what blocked you? |
Time was a blocker. |
https://github.com/mcollina/syncthrough offers 3x the throughput of
Transform
. https://www.npmjs.com/package/minipass offer similar characteristics.I think bringing in a faster set of Streams that enforce backpressure would speed up an area of Node.js that is showing its time.
The text was updated successfully, but these errors were encountered: