Skip to content

[Player Events] Zone Fetch Settings from QS if Enabled#4915

Merged
Akkadius merged 3 commits intomasterfrom
akkadius/pull-player-event-settings-qs
Jun 25, 2025
Merged

[Player Events] Zone Fetch Settings from QS if Enabled#4915
Akkadius merged 3 commits intomasterfrom
akkadius/pull-player-event-settings-qs

Conversation

@Akkadius
Copy link
Copy Markdown
Contributor

Description

When large servers have high player event log volume, they can choose to offload player events to another server (QueryServ) where it has a separate logging database and server entirely.

The problem with this is that player event log settings can become split brained because the settings are authoritative on QS (log server). The zone server loads settings locally instead of pulling them from QS. This PR addresses that by having QS send settings when zone initially connects.

When #reload or Spire reload requests are issued, when the rule Logging:PlayerEventsQSProcess is enabled, zones will instead reload from QS instead of reloading from their local.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Testing

When zones boot, they fetch settings from QS when Logging:PlayerEventsQSProcess is enabled

image

This is when log settings are reloaded either from in game or from Spire

image

Checklist

  • I have tested my changes
  • I have performed a self-review of my code. Ensuring variables, functions and methods are named in a human-readable way, comments are added only where naming of variables, functions and methods can't give enough context.
  • I own the changes of my code and take responsibility for the potential issues that occur

@Akkadius Akkadius merged commit fb4d003 into master Jun 25, 2025
2 checks passed
@Akkadius Akkadius deleted the akkadius/pull-player-event-settings-qs branch June 25, 2025 17:21
@Akkadius Akkadius mentioned this pull request Jun 25, 2025
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.

3 participants