Skip to content

perf(group): Cache group membership in distributed cache - #62581

Open
CarlSchwan wants to merge 4 commits into
masterfrom
carl/cache-group-membership
Open

CarlSchwan wants to merge 4 commits into
masterfrom
carl/cache-group-membership

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

And move the group management listener registration in the core app instead of the files_sharing app (which can be disabled).

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@CarlSchwan CarlSchwan added this to the Nextcloud 35 milestone Jul 27, 2026
@CarlSchwan CarlSchwan self-assigned this Jul 27, 2026
@CarlSchwan
CarlSchwan requested a review from a team as a code owner July 27, 2026 13:42
@CarlSchwan
CarlSchwan requested review from Altahrim, icewind1991, provokateurin and salmart-dev and removed request for a team July 27, 2026 13:42
@CarlSchwan
CarlSchwan force-pushed the carl/cache-group-membership branch 6 times, most recently from 056a23d to 3dca2f3 Compare July 31, 2026 08:40
Comment thread lib/private/App/AppManager.php Outdated
Comment thread lib/private/Group/Manager.php Outdated
Comment thread lib/private/SubAdmin.php Outdated
This was referenced Aug 12, 2026
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 20, 2026
@skjnldsv skjnldsv mentioned this pull request Aug 25, 2026
And move the group management listener registration in the core app
instead of the files_sharing app (which can be disabled).

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Not used anymore in github.com/nextcloud and there is a modern
replacement since NC 18

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan
CarlSchwan force-pushed the carl/cache-group-membership branch from 3dca2f3 to d85b052 Compare October 2, 2026 10:24
@CarlSchwan
CarlSchwan force-pushed the carl/cache-group-membership branch from d85b052 to f176b7c Compare October 2, 2026 10:25
Also invalidate the group membership cache after membership changes so
that a concurrent request can't keep stale data in the distributed cache.

Assisted-by: ClaudeCode:claude-opus-5-5
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan
CarlSchwan force-pushed the carl/cache-group-membership branch from f176b7c to 5ebeade Compare October 2, 2026 10:28

This branch has not been deployed

No deployments
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.

4 participants