Skip to content

Adds documentation on manual span propogation #153

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

Merged
merged 3 commits into from
Jun 18, 2025

Conversation

NeedleInAJayStack
Copy link
Contributor

Motivation:

I work with a number of packages that are heavily NIO-based. From the current documentation, I found it unclear how to create nested spans using this package in that environment where Task local variables were not preserved throughout EventLoopFuture chains.

Modifications:

This simply adds documentation for how to manually propogate spans by providing context arguments. It offers some examples that clarify that those contexts should come from started spans in order for trace IDs to be linked.

Result:

No functional changes, simply more thorough documentation.

Please let me know if I missed something or you have different suggestions!

**Motivation:**

I work with a number of packages that are heavily NIO-based. I found it unclear how to create nested spans using this package in that environment where Task local variables were not preserved throughout EventLoopFuture chains.

**Modifications:**

This simply adds documentation for how to manually propogate spans by providing context arguments. It offers some examples that clarify that those contexts should come from started spans in order for trace IDs to be linked.

**Result:**

No functional changes, simply more thorough documentation.
Copy link
Member

@ktoso ktoso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I missed this, I think it's a good addition

@ktoso ktoso modified the milestone: 1.2.1 Jun 18, 2025
@ktoso ktoso added semver/none No version bump required. labels Jun 18, 2025
@ktoso ktoso merged commit b787967 into apple:main Jun 18, 2025
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver/none No version bump required.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants