Comments (4)
I'm not sure why that isn't the behavior.
from telemetry.
We don't perform any merging, so indeed the docs are inaccurate. PRs welcome!
from telemetry.
Hello! I'm still a tad confused about this, even after the linked PR got merged.
In the quoted documentation paragraph, the docs mention that a reporter should be able to rely entirely on the stop event. However, this doesn't seem possible if StartMetadata isn't merged (automatically) into StopMetadata, because there is possibly a large amount of metadata that is unavailable to the stop event by default, and must be manually merged into the stop metadata when a span
wrapper is implemented.
This seems to me that it makes it very easy to implement telemetry in a way which makes reporters very arduous to write (with need for long-running state to track and match span
-related events via their references). I'm unsure if this is the intent, or if the documentation is incorrect, or if I'm just misunderstanding things entirely.
from telemetry.
However, this doesn't seem possible if StartMetadata isn't merged (automatically) into StopMetadata,
we should document that it is extremely Recommended for callers to return the stop metadata as a superset of the start metadata.
from telemetry.
Related Issues (20)
- Can't start app in Mix release {:bad_return, {{:telemetry_app, :start... HOT 10
- Making execution order of handlers guaranteed HOT 16
- Is telemetry distributed ? HOT 2
- Automatically detach when attaching process ends HOT 1
- Using telemetry as a dependency in an Elixir script HOT 11
- Support a debug mode and capture unhandled events in debug mode HOT 2
- Safe ETS interactions HOT 6
- Errors in readme HOT 4
- Static API for discovering events HOT 28
- Add telemetry:span/3 HOT 67
- A flaky and odd compilation error HOT 16
- Guidance around monotonic times HOT 5
- Nested payload for measurement? HOT 1
- Release 0.4.2 HOT 3
- What's the difference with gen_event? HOT 1
- Shall we release 1.0? HOT 5
- span needs to pass context to the stop event HOT 16
- span/3 StartMetadata and StopMetadata semantics HOT 1
- telemetry.beam failed: :badfile HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from telemetry.