Skip to content

[Database] Add Indexes to NPC's Spawns Loot#4972

Merged
Kinglykrab merged 1 commit intomasterfrom
akkadius/add_indexes_npc_spawns_loot
Jul 30, 2025
Merged

[Database] Add Indexes to NPC's Spawns Loot#4972
Kinglykrab merged 1 commit intomasterfrom
akkadius/add_indexes_npc_spawns_loot

Conversation

@Akkadius
Copy link
Copy Markdown
Contributor

@Akkadius Akkadius commented Jul 27, 2025

Description

Adds indexes to a few content tables that is used for newly developed database browser tooling.

Asked for by Rekka in Discord

Type of change

  • Performance

Testing

Query plan before

image

After

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
  • If my changes make database schema changes, I have tested the changes on a local database (attach image). Updated version.h CURRENT_BINARY_DATABASE_VERSION to the new version. (Delete this if not applicable)

@Kinglykrab Kinglykrab merged commit f3232cd into master Jul 30, 2025
2 checks passed
@Kinglykrab Kinglykrab deleted the akkadius/add_indexes_npc_spawns_loot branch July 30, 2025 13:37
joligario added a commit to ProjectEQ/peqphpeditor that referenced this pull request Jul 31, 2025
@Akkadius Akkadius mentioned this pull request Aug 3, 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