Skip to content
This repository has been archived by the owner on Feb 26, 2021. It is now read-only.

Migration to Pull Streams - Experiment B #20

Closed
wants to merge 5 commits into from
Closed

Conversation

daviddias
Copy link
Member

@daviddias daviddias commented Aug 28, 2016

Mode B

This is one of the experiments done in order to test the migration to pull-streams, with regards to stream muxing.

Mode B consists in having pull streams exposed from the spdy stream muxer, while layering spdy on top of Node.js Streams. This experiment was created to understand if the problem found come from the above layering or the bottom layering, it showed that the bottom layered is causing the problem, but also demonstrated that the wrapping is more memory intensive.

Illustration:

image

Notes

  • Tests pass
  • Mega stress test runs out of memory if not throttled

How to test it yourself

In order to test this, you need to have to have the pull-streams migration of interface-connection available in your local env. Here are the steps

> git clone git@github.com:libp2p/interface-connection.git -b pull
> cd interface-connection
> npm i && npm link
> cd ..
> git clone git@github.com:libp2p/interface-stream-muxer.git -b pull-mode-b
> cd interface-stream-muxer
> npm i && npm link
> cd ..
> git clone git@github.com:libp2p/js-libp2p-spdy.git -b pull-mode-b
> cd js-libp2p-spdy 
> npm i
> npm link interface-connection
> npm link interface-stream-muxer
> npm test

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants