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

Don't trigger LobbyInfoSynced on ping update. #15182

Merged
merged 1 commit into from May 30, 2018

Conversation

Projects
None yet
3 participants
@pchote
Copy link
Member

pchote commented May 28, 2018

This interface is only used for updating the master server advertisement, which doesn't use the player pings at all. Removing this should reduce the master server traffic by a factor of several tens.

Don't trigger LobbyInfoSynced on ping update.
This interface is only used for updating the master
server advertisement, which doesn't use the player
pings at all.  Removing this should reduce the
master server traffic by a factor of several tens.

@pchote pchote added this to the Next release milestone May 28, 2018

@GraionDilach
Copy link
Contributor

GraionDilach left a comment

👍

@reaperrr reaperrr merged commit 6e400c5 into OpenRA:bleed May 30, 2018

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@pchote pchote deleted the pchote:reduce-masterserver-load branch Jul 9, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.