Skip to content

docs: document session lock event timing and session info - #6205

Merged
peholmst merged 4 commits into
mainfrom
doc-bot/vaadin-flow/25964-ae5732d1be391097
Nov 9, 2026
Merged

peholmst merged 4 commits into
mainfrom
doc-bot/vaadin-flow/25964-ae5732d1be391097

Conversation

@vaadin-bot

@vaadin-bot vaadin-bot commented Sep 29, 2026 •

Copy link
Copy Markdown
Contributor

Documentation for vaadin/flow#25964 by @totally-not-ai[bot].

Note

The source pull request is merged, so this documentation describes the final
shape of the change. Please review it and mark it ready for review.

Change categories: NEW_FEATURE, API_CHANGE, BEHAVIOR_CHANGE

File Change
articles/flow/advanced/session-lock-and-rpc-listeners.adoc Document AbstractSessionLockEvent.getSession(), SessionLockAcquiredEvent.getWaitTime(), and SessionLockReleasedEvent.getHoldTime(); replace the outdated statement that a ThreadLocal is needed to measure lock timing; note that a release listener only fires for a hold it was already registered for when the lock was acquired.
src/main/java/com/vaadin/demo/flow/advanced/eventbus/SessionLockMetricsInitListener.java Rewrite the example listener to read getWaitTime()/getHoldTime() directly instead of tracking timestamps in ThreadLocal fields.

Auto-generated by the Documentation Bot — review before merging.

Generated by Documentation Bot for #25964 · agent · 113.7 AIC · ⌖ 6.98 AIC · ⊞ 11.7K · ◷

  • expires on Oct 29, 2026, 12:54 PM UTC

Working on behalf of Petter

Describe SessionLockAcquiredEvent.getWaitTime(),
SessionLockReleasedEvent.getHoldTime(), and
AbstractSessionLockEvent.getSession() on the Service Event Bus page,
and update the example listener to use them instead of timing the
events by hand. Also note that a release listener only fires for
holds it was registered before.

Documents vaadin/flow#25964 (`ce47866e3db8021814f33162c71bafdf980b04ee`).
@github-actions

github-actions Bot commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

Preview Deployment

This PR has been deployed for preview.

URL: https://docs-preview-pr-6205.fly.dev

Changed pages

Added content is highlighted in green; removed content is marked in red on each page.

Built from b8bd7b8

@totally-not-ai
totally-not-ai Bot marked this pull request as ready for review November 7, 2026 09:01
@peholmst
peholmst merged commit b2c2aba into main Nov 9, 2026
13 checks passed
@peholmst
peholmst deleted the doc-bot/vaadin-flow/25964-ae5732d1be391097 branch November 9, 2026 08:07
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