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

send "ingame" status change only to players which are inside the same battle #111

Open
abma opened this issue Oct 25, 2014 · 9 comments
Open

Comments

@abma
Copy link
Contributor

abma commented Oct 25, 2014

no need to send it to all clients

@sprunk
Copy link

sprunk commented Oct 26, 2014

it's useful to know whether someone is ingame or not, even if you're not in the same battle, eg. if you need to talk to a mod or a clan-mate you can pick one that is not ingame

@abma
Copy link
Contributor Author

abma commented Oct 26, 2014

performance vs comfort

sending status to every connected client is VERY inefficient

i guess friend list should be used for that as well.

@gajop
Copy link
Member

gajop commented Oct 30, 2014

Should also be sent to friends automatically, and should be able to query a user's status (like it works in most systems).

@PicassoCT
Copy link

Sorry, but i still think, the concerns would be seperated in a most clean fashion, if the client registers for the data it wants to hear and has to take the burden of concern.
Else you will have a giant state machine on the server, for every client to find out what data he needs.
Finally with over 9000 players, there comes a point where you have to add multi-server support - and then what? Handing over a client-concern-simulation with all the friends-lists etc. sounds at lot more complicated, then just handing down a list of listeners and add what is still missing to one big server-server listener to transfer info the other server just doesent have.
But i may be wrong. Its just instinct yelling, that the object that has the most agency with something, should have the burden of concern.

@silentwings
Copy link
Contributor

silentwings commented Aug 31, 2018

Perhaps its enough just to send the ingame status for battle hosts

@sprunk
Copy link

sprunk commented Sep 3, 2018

Sounds like a fairly good approximation if 100% accuracy is not required.

@silentwings
Copy link
Contributor

closing for now - can be reopened in future if traffic reduction is needed

@abma
Copy link
Contributor Author

abma commented Feb 7, 2019

this request is not about traffic reduction: its just not needed and also it scales very badly.

@abma abma reopened this Feb 7, 2019
@abma
Copy link
Contributor Author

abma commented Feb 7, 2019

as compromise only the ingame status of hosts could be send.

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

No branches or pull requests

5 participants