vaadin / vaadin/observability-kit
Triage which remaining 4.1 spans come back before 5.0
Nobody has claimed this yet.
- Dominant language
- Java
- Stars
- 7
- Forks
- 5
- Avg merge
- 2d 16h
- Merged PRs (30d)
- 25
Description
5.0 is not released, so no 4.1 span has actually been dropped. The consolidation decisions made during the rewrite were never reviewed as decisions, and this issue closes them out so that every 4.1 span has an explicit answer rather than an implicit omission.
Starting position, for argument:
Worth restoring
Component creation(ComponentRenderer). A heavy per-row renderer is a common cause of a slow Grid, so this has real diagnostic value. Not as one span per cell, which would be trace spam. One span per render pass plus a timer tagged by component class. This needs a Flow SPI, so realistically after GA. Prior art: #362 already tracks restoring renderer cost instrumentation, so the decision here should land there rather than fork.- A
vaadin.viewattribute onvaadin.rpc. 4.1 had it. Todayrouteexists only on navigation spans, so an RPC cannot be attributed to a view. Cheap and fits in GA.
Proposed to keep consolidated
- Static resources, PWA resources and web component bootstrap. Low diagnostic value, and usually served by a CDN or reverse proxy in production.
type=staticalready covers them. SessionRequestHandler.*andUnsupportedBrowserHandler.*. Negligible value.- Per-handler RPC span names (
Event:,Sync:,Invoke server method:,Attach existing element,Handle return channel,Handle navigation).vaadin.rpcwithtype,vaadin.rpc.eventandvaadin.rpc.componentcarries more information than the 4.1 span names did.
Already covered
Heartbeat, viatype=heartbeat.
Tracked separately
Hilla endpoint spans and browser spans are scope decisions of their own, not part of this triage. Note that upgrading Hilla applications lose their endpoint spans with no replacement, which needs a release note line regardless of the outcome here.
Background: docs/spans-parity-plan.md.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with docs/spans-parity-plan.md and compare its 4.1 spans with the decisions listed here. Review #362 for the renderer instrumentation overlap, then document explicit outcomes for each span group and identify the separate Hilla and browser scope decisions, including the release-note requirement.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- observability
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Active
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100