Skip to content

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