Skip to content

perf(db): Optimize jobs table indexes#51857

Draft
ChristophWurst wants to merge 3 commits intomasterfrom
perf/db/jobs-table-indexes
Draft

perf(db): Optimize jobs table indexes#51857
ChristophWurst wants to merge 3 commits intomasterfrom
perf/db/jobs-table-indexes

Conversation

@ChristophWurst
Copy link
Member

Summary

  • Drop redundant index on class
  • Extend index on time_sensitive to be more useful

For existing installations we will only drop the redundant index. The replacement will be done by the admin via occ. This avoids long downtime and times without the new index.
For new installations the index will exist out of the box.

TODO

  • Do

Checklist

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
Nextcloud commonly selects a job by looking at all three columns.

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst ChristophWurst self-assigned this Apr 2, 2025
@ChristophWurst ChristophWurst requested a review from a team as a code owner April 2, 2025 07:46
@ChristophWurst ChristophWurst requested review from come-nc and removed request for a team April 2, 2025 07:46
Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
@ChristophWurst
Copy link
Member Author

image

time_sensitive isn't as selective as last_checked, so I flipped the order in 9cac577.

I'm setting this PR to draft until we have the best results from production.

@ChristophWurst ChristophWurst marked this pull request as draft April 2, 2025 08:12
@ChristophWurst ChristophWurst added 2. developing Work in progress and removed 3. to review Waiting for reviews labels Apr 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: Obsolete index jobs.job_class_index

1 participant