Skip to content

perf: only load possible results in UserMountCache::getMountForPath#59649

Open
icewind1991 wants to merge 2 commits intomasterfrom
usermountcache-get-by-path-opt
Open

perf: only load possible results in UserMountCache::getMountForPath#59649
icewind1991 wants to merge 2 commits intomasterfrom
usermountcache-get-by-path-opt

Conversation

@icewind1991
Copy link
Copy Markdown
Member

Extracted from #57760 for easier reviews.

Instead of always loading all mounts for the user and then finding the relevant one for a path. We first collect the possible paths the mountpoint can be at (the path and all it's parents), and then only load the mounts for those paths.

Signed-off-by: Robin Appelman <robin@icewind.nl>
Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991 icewind1991 added this to the Nextcloud 34 milestone Apr 15, 2026
@icewind1991 icewind1991 requested a review from a team as a code owner April 15, 2026 14:01
@icewind1991 icewind1991 requested review from Altahrim, leftybournes, nfebe and salmart-dev and removed request for a team April 15, 2026 14:01
@icewind1991
Copy link
Copy Markdown
Member Author

/backport to stable33

@icewind1991
Copy link
Copy Markdown
Member Author

/backport to stable32

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants