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

split matchmakers from intent gossiper #763

Merged
merged 11 commits into from
Jan 31, 2022
Merged

Conversation

tzemanovic
Copy link
Member

@tzemanovic tzemanovic commented Jan 3, 2022

closes #579

currently based on #754 rebased on master.

Note that there is no connection re-try on the client side atm, so an intent gossiper node must be running before the matchmaker(s) that want to connect to it and on lost connection the matchmakers shut-down. I've opened #818 to improve this

@tzemanovic tzemanovic force-pushed the tomas/matchmaker-gossip-channel branch 2 times, most recently from 38cc169 to d41a85c Compare January 14, 2022 12:12
@tzemanovic tzemanovic force-pushed the tomas/matchmaker-gossip-channel branch 2 times, most recently from 9531de3 to 1e0c79b Compare January 24, 2022 10:44
@tzemanovic tzemanovic force-pushed the tomas/matchmaker-gossip-channel branch from 1e0c79b to fd6f1e5 Compare January 24, 2022 10:55
@tzemanovic tzemanovic force-pushed the tomas/matchmaker-gossip-channel branch from fd6f1e5 to 9252137 Compare January 24, 2022 12:29
@tzemanovic
Copy link
Member Author

pls update wasm

@tzemanovic tzemanovic marked this pull request as ready for review January 24, 2022 14:10
@juped juped self-assigned this Jan 24, 2022
@tzemanovic tzemanovic force-pushed the tomas/matchmaker-gossip-channel branch from 7d4b0c5 to e5a7548 Compare January 28, 2022 11:22
@tzemanovic
Copy link
Member Author

pls update wasm

@tzemanovic tzemanovic force-pushed the tomas/matchmaker-gossip-channel branch from 293b4ca to 281c3ef Compare January 28, 2022 15:33
@juped juped merged commit 9edc38f into master Jan 31, 2022
@juped juped deleted the tomas/matchmaker-gossip-channel branch January 31, 2022 17:26
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.

Create separate command for running the matchmaker
3 participants