Skip to content

[stable32] perf(sharing): Avoid loading all shares from all users when unsharing#58659

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

[stable32] perf(sharing): Avoid loading all shares from all users when unsharing#58659
backportbot[bot] wants to merge 2 commits intostable32from
backport/58057/stable32

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Mar 2, 2026

Backport of #58057

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

Todo

  • Review and resolve any conflicts
  • 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.

First check which users have a shares and for which providers and then
only load these shares.

Avoid doing at most 5 SQL queries for each users a share was shared with if
there are no shares.

Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan force-pushed the backport/58057/stable32 branch from e062e97 to cc38241 Compare March 2, 2026 11:39
@CarlSchwan CarlSchwan marked this pull request as ready for review March 2, 2026 11:39
@CarlSchwan CarlSchwan requested a review from a team as a code owner March 2, 2026 11:39
@CarlSchwan CarlSchwan requested review from leftybournes and nfebe and removed request for a team March 2, 2026 11:39
@CarlSchwan CarlSchwan force-pushed the backport/58057/stable32 branch 2 times, most recently from 1333d9b to 543dacf Compare March 2, 2026 13:09
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan force-pushed the backport/58057/stable32 branch from 543dacf to c3a15af Compare March 2, 2026 15:08
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