Skip to content

Fix/setup for path optimizations#58325

Draft
salmart-dev wants to merge 5 commits intofix/mount-manager-improve-get-by-providerfrom
fix/setup-for-path-optimizations
Draft

Fix/setup for path optimizations#58325
salmart-dev wants to merge 5 commits intofix/mount-manager-improve-get-by-providerfrom
fix/setup-for-path-optimizations

Conversation

@salmart-dev
Copy link
Contributor

Summary

setupForPath has a few issues slowing things down:

  1. it is currently hammering the oc_mounts table because of the call to getMountForPath which is always executed unless the user or the provider have been fully set up
  2. keeps being re-executed if called with $forChildren = false for paths whose parents have not been set up with children and the path is not an MP itself (because we mark the MP as setup, not the path)
  3. suffers from the issue that if any code calls setupForPath('/Talk/01'), setupForPath('/Talk/02'), ... setupForPath('/Talk/100') then it would cause N+1 setups

Checklist

Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
Signed-off-by: Salvatore Martire <4652631+salmart-dev@users.noreply.github.com>
@salmart-dev salmart-dev self-assigned this Feb 13, 2026
@salmart-dev salmart-dev changed the base branch from master to fix/mount-manager-improve-get-by-provider February 13, 2026 11:22
@salmart-dev salmart-dev force-pushed the fix/mount-manager-improve-get-by-provider branch from 4987175 to d400dd4 Compare February 13, 2026 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant