Skip to content

Add support for PubSub#20

Open
Dunedan wants to merge 3 commits into0ad:masterfrom
Dunedan:pubsub
Open

Add support for PubSub#20
Dunedan wants to merge 3 commits into0ad:masterfrom
Dunedan:pubsub

Conversation

@Dunedan
Copy link
Copy Markdown
Collaborator

@Dunedan Dunedan commented Apr 22, 2023

This adds support for publishing game lists, rating lists and the leader board via PubSub. With matching support in pyrogenesis this will improve the performance and efficiency of the lobby, as the bots then don't have to manually send updates to each connected client, but the clients subscribe to the PubSub nodes and the XMPP server takes care of deliverying updates instead.

The previous behavior of sending stanzas with updates to every connected client is still available, but can be disabled with the --disable-legacy-lists command line flag. The performance benefits of PubSub do only materialize when the legacy behavior is disabled.

This adds support for publishing game lists, rating lists and the
leader board via PubSub. With matching support in pyrogenesis this will
improve the performance and efficiency of the lobby, as the bots then
don't have to manually send updates to each connected client, but the
clients subscribe to the PubSub nodes and the XMPP server takes care of
deliverying updates instead.

The previous behavior of sending stanzas with updates to every connected
client is still available, but can be disabled with the
`--disable-legacy-lists` command line flag. The performance benefits
of PubSub do only materialize when the legacy behavior is disabled.
The disco query to list existing PubSub nodes on startup was missing for
EcheLOn.
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.

1 participant