-
-
Notifications
You must be signed in to change notification settings - Fork 103
Add FedifySpanExporter for distributed trace storage
#502
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
Changes from all commits
Commits
Show all changes
18 commits
Select commit
Hold shift + click to select a range
9f49d86
Add FedifySpanExporter for distributed trace storage
dahlia 7964cb3
Add PR #502 reference to CHANGES.md
dahlia 31edf2a
Use event.time for accurate timestamps in TraceActivityRecord
dahlia d9aea3f
Fix race condition in #updateTraceSummary with CAS
dahlia f6e11e8
Fix getRecentTraces sorting/limiting logic
dahlia f5b7876
Fix fragile _summaries key filtering in getActivitiesByTraceId
dahlia be8654c
Add error logging in FedifySpanExporter.export()
dahlia 4e1e573
Fix misleading forceFlush() comment
dahlia ccf4dba
Document httpSignaturesKeyId as optional
dahlia 4bcfdf2
Extract #setWithCasRetry helper to reduce duplication
dahlia 0093973
Use immutable array operation in #storeWithCas
dahlia cb83e02
Remove unnecessary _summaries filter in getActivitiesByTraceId
dahlia eff3acc
Use Promise.allSettled() in #exportAsync for robust error handling
dahlia 56b46d4
Treat empty httpSignaturesKeyId as undefined
dahlia 7136157
Document list() requirement for getRecentTraces()
dahlia f921cca
Fix misleading test description for shutdown()
dahlia e2a1807
Use AggregateError for multiple export failures
dahlia 85d7c09
Extract test values into variables for better maintainability
dahlia File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.