Skip to content

Conversation

@provokateurin
Copy link
Member

@provokateurin provokateurin commented May 5, 2025

Summary

The providers didn't have an efficient way to query all shares in a folder without filtering by user and the current solution which was doing it for each node in the folder caused performance issues.

Checklist

@provokateurin provokateurin added this to the Nextcloud 32 milestone May 5, 2025
@provokateurin provokateurin requested a review from icewind1991 May 5, 2025 13:10
@provokateurin provokateurin requested a review from a team as a code owner May 5, 2025 13:10
@provokateurin provokateurin requested review from sorbaugh and yemkareems and removed request for a team May 5, 2025 13:10
@provokateurin provokateurin added 3. to review Waiting for reviews pending documentation This pull request needs an associated documentation update performance 🚀 labels May 5, 2025
@provokateurin
Copy link
Member Author

/backport to stable31

@provokateurin provokateurin requested review from artonge and come-nc May 5, 2025 13:11
…filtering by user

Signed-off-by: provokateurin <kate@provokateurin.de>
@provokateurin provokateurin force-pushed the perf/share20/get-all-shares-in-folder branch from 1c180b5 to ae16a28 Compare May 8, 2025 13:30
@provokateurin provokateurin enabled auto-merge May 8, 2025 13:30
@provokateurin provokateurin merged commit 648745d into master May 8, 2025
198 of 204 checks passed
@provokateurin provokateurin deleted the perf/share20/get-all-shares-in-folder branch May 8, 2025 14:24
@provokateurin provokateurin removed the pending documentation This pull request needs an associated documentation update label May 15, 2025
@skjnldsv skjnldsv mentioned this pull request Aug 19, 2025
@skjnldsv skjnldsv modified the milestones: Nextcloud 32, Nextcloud 33 Sep 28, 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.

Performance regression in 31 with co-owner shares

5 participants