Skip to content

Add Jaeger tracing for waitForBookmark storage API#5622

Merged
justin-mp merged 1 commit intomainfrom
jmp/waitForBookmark-tracing
Dec 2, 2025
Merged

Add Jaeger tracing for waitForBookmark storage API#5622
justin-mp merged 1 commit intomainfrom
jmp/waitForBookmark-tracing

Conversation

@justin-mp
Copy link
Copy Markdown
Contributor

Add tracing span creation for waitForBookmark at the DurableObjectStorage API layer. Update ActorCacheInterface and ActorSqlite to accept SpanParent parameter for trace context propagation.

🤖 Generated with Claude Code

@justin-mp justin-mp requested review from a team as code owners December 1, 2025 19:06
@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Dec 1, 2025

CodSpeed Performance Report

Merging #5622 will not alter performance

Comparing jmp/waitForBookmark-tracing (daeafd5) with main (fa09e00)

Summary

✅ 57 untouched
⏩ 30 skipped1

Footnotes

  1. 30 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

Add tracing span creation for waitForBookmark at the DurableObjectStorage
API layer. Update ActorCacheInterface and ActorSqlite to accept SpanParent
parameter for trace context propagation.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@justin-mp justin-mp force-pushed the jmp/waitForBookmark-tracing branch from a81a613 to daeafd5 Compare December 2, 2025 11:18
@justin-mp justin-mp merged commit a160b37 into main Dec 2, 2025
31 of 33 checks passed
@justin-mp justin-mp deleted the jmp/waitForBookmark-tracing branch December 2, 2025 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants