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

Intention of deprecation #90

Open
vasco-santos opened this issue Nov 27, 2019 · 4 comments
Open

Intention of deprecation #90

vasco-santos opened this issue Nov 27, 2019 · 4 comments

Comments

@vasco-santos
Copy link
Member

In the context of ipfs/js-ipfs#1670, we have been refactoring all libp2p codebase to use async await, async iterators, as well as working on general API improvements to have a better and easier to use js-libp2p.

Taking into account that spdy is an outdated protocol, there are no reasons to use it instead of libp2p-mplex and from our knowledge there are no active projects using libp2p-spdy, we have the intention of deprecating libp2p-spdy and use this time in enhancing the important pieces of libp2p.

If anyone is using this module and want to help us refactoring it, so that you and anyone else interested can use it, let us know. We are happy to help with it too

@jacobheun
Copy link
Contributor

cc @libp2p/repos-javascript

@lidel lidel pinned this issue Nov 27, 2019
@vasco-santos
Copy link
Member Author

cc

@mkg20001 @jacogr @stefanionescu @richard-ramos @fredfortier @lacabra @notadd

@daviddias
Copy link
Member

@vasco-santos before deprecating, can you make sure to benchmark spdy vs. multiplex. I remember that spdy was way more performant in the past.

@vasco-santos
Copy link
Member Author

vasco-santos commented Nov 28, 2019

@vasco-santos before deprecating, can you make sure to benchmark spdy vs. multiplex. I remember that spdy was way more performant in the past.

@daviddias I think we would not get really accurate results, since to have the exact same conditions we would need to refactor spdy to leverage all the other changes we did that also improved beformance for mplex

Regarding past performance, do you have any more recent conclusions @jacobheun ?

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

No branches or pull requests

3 participants