8000 Update outdated CacheManager revision comment by ahus1 · Pull Request #55018 · keycloak/keycloak · GitHub
[go: up one dir, main page]

Skip to content

Update outdated CacheManager revision comment - #55018

Draft
ahus1 wants to merge 3 commits into
keycloak:mainfrom
ahus1:is-45080-cachemanager-comment
Draft

ahus1 wants to merge 3 commits into
keycloak:mainfrom
ahus1:is-45080-cachemanager-comment

Conversation

@ahus1
@ahus1 ahus1 commented Nov 10, 2026 •
Copy link
Copy Markdown
Member

Closes #45080

The comment described a marker-key mechanism (invalidation.key.<userid>) that the code doesn't implement. It is replaced by ordered steps describing the actual flow: invalidateObject plus a cluster InvalidationEvent bumping the revision on every node, and addRevisioned rejecting the stale entry.

Closes keycloak#45080

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Alexander Schwartz <alexander.schwartz@ibm.com>
ahus1 and others added 2 commits November 10, 2026 20:22
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Alexander Schwartz <alexander.schwartz@ibm.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: Alexander Schwartz <alexander.schwartz@ibm.com>
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.

Invalid/Outdated comments of CacheManager

1 participant

0