Skip to content

perf(FsEventScheduler): Make sure to get node owner from caller, if possible#206

Merged
marcelklehr merged 2 commits intomainfrom
perf/no-unnecessary-owner-check
Jan 27, 2026
Merged

perf(FsEventScheduler): Make sure to get node owner from caller, if possible#206
marcelklehr merged 2 commits intomainfrom
perf/no-unnecessary-owner-check

Conversation

@marcelklehr
Copy link
Member

fixes #205

…ossible

Signed-off-by: Marcel Klehr <mklehr@gmx.net>
Signed-off-by: Marcel Klehr <mklehr@gmx.net>
@oleksandr-nc
Copy link

And maybe ownerId can me renamed to userId.
AFAIK, for the Group Folders mounts it's really the "user who has access" rather than an owner - but maybe I am wrong

@marcelklehr marcelklehr merged commit 8cdf6d0 into main Jan 27, 2026
36 checks passed
@marcelklehr marcelklehr deleted the perf/no-unnecessary-owner-check branch January 27, 2026 10:36
@kyteinsky kyteinsky mentioned this pull request Feb 5, 2026
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.

FsEventScheduler::onAccessUpdateDecl does a needless getOwnerForFileId

2 participants