-
Notifications
You must be signed in to change notification settings - Fork 86
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
adding baggage to whitepaper #202
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for tag-observability ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Signed-off-by: Guangya Liu <[email protected]>
@@ -259,6 +260,17 @@ Profiling data produced by runtimes typically includes statistics down to the li | |||
|
|||
> Image 2 shows an example icicle graph for a CPU profile of an application written in Go. It includes a kernel operation done by the Syscall call. This profile highlights that 35% of CPU time was used to compute hash, indicating potential optimization opportunities. [Source](https://pprof.me/9ce2c2d/). | |||
|
|||
### Baggage |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder, how is that a new signal vs trace detail? 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@bwplotka can you check https://opentelemetry.io/docs/concepts/signals/baggage/ ? From the documentation there, the baggage
is a new signal at same level as trace, metrics and logs.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree with @bwplotka
Though Baggage is at the same level as traces, metrics, and logs on the site, one of the first paragraphs mentions that Baggage is transferred alongside Context which can be attached to traces, metrics, or logs. Given it attaches to those three signal types, Baggage isn't separate from them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Some question first
No description provided.