Skip to content
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

Add distributed store support. #46

Closed
wants to merge 1 commit into from

Conversation

tsujio
Copy link

@tsujio tsujio commented Jun 8, 2014

In this patch PeerServers compose a cluster and they forward client's message to the other PeerServer if the dst of the message is not found in the local _clients property.

PeerServers find which server to forward messages by memcached shared by the cluster.

@jure
Copy link

jure commented Jun 19, 2014

Wow! Are you kidding me?! This is fantastic! Great work, @tsujio!

Given that one of the big limitations for scalability for self-hosted PeerJS is the number of WebSocket connections each individual server can handle, this is a great step towards removing that obstacle.

Not really sure if this belongs directly to PeerJS server, but this is an awesome use-case for extensions.

@michelle
Copy link
Member

Sorry for the delay! Looking at this now :).

@michelle
Copy link
Member

Awesome work! I'm ==@jure that this likely doesn't belong in peerjs-server master, but would love to consider ways to make the server more extensible for things like this (let me know if you have any thoughts!).

I'm going to play around with this implementation a bit to see how it could be rewritten as an extension (and what changes to the PeerJS server are necessary).

Thanks again!

@ummjackson
Copy link

Unfortunately this still centralizes around shared memcached instances... would be great to have some sort of mesh-database system going between PeerServers without the need for sharing memcached.

@jure
Copy link

jure commented Sep 29, 2014

This is more about scaling than it is about decentralization. For a maximally decentralized network (given the signaling constraints), take a look https://github.com/boplish/core or more specifically boplish/core#4.

@Iragne
Copy link

Iragne commented Jan 1, 2015

very interesting If we can merge this request in the master ??

@afrokick afrokick closed this Apr 1, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants