Skip to content

[stable32] Optimize cleanup job#8444

Open
backportbot[bot] wants to merge 2 commits intostable32from
backport/8421/stable32
Open

[stable32] Optimize cleanup job#8444
backportbot[bot] wants to merge 2 commits intostable32from
backport/8421/stable32

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot bot commented Apr 8, 2026

Backport of #8421

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Do that in one query instead of ending up doing N+1 queries.

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@backportbot backportbot bot added bug Something isn't working 3. to review performance 🚀 labels Apr 8, 2026
@backportbot backportbot bot added this to the Nextcloud 32.0.9 milestone Apr 8, 2026
Copy link
Copy Markdown
Member

@nickvergessen nickvergessen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This breaks daily and other master/main instances that run pgsql at least.
https://github.com/nextcloud/text/pull/8421/changes#r3055845268

@CarlSchwan CarlSchwan force-pushed the backport/8421/stable32 branch from ead35bd to 24f20a6 Compare April 10, 2026 12:42
@CarlSchwan CarlSchwan marked this pull request as ready for review April 10, 2026 12:42
Decrease time to execute the query from 10s of seconds to a few
miliseconds.

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan force-pushed the backport/8421/stable32 branch from 24f20a6 to b7e5793 Compare April 10, 2026 12:43
@nickvergessen nickvergessen dismissed their stale review April 10, 2026 12:56

Addressed

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 10, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@CarlSchwan CarlSchwan enabled auto-merge April 10, 2026 14:14
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.

2 participants