Skip to content

Comments

add tracing spans#82

Closed
jgodlew wants to merge 6 commits intomainfrom
jgodlew/core-tracing
Closed

add tracing spans#82
jgodlew wants to merge 6 commits intomainfrom
jgodlew/core-tracing

Conversation

@jgodlew
Copy link
Contributor

@jgodlew jgodlew commented Nov 2, 2024

No description provided.

jgodlew added a commit that referenced this pull request May 2, 2025
* Creates a session_id whenever the `data_client` is used or a
FileUploadSession/FileDownloader is created to be propagated to the new
remote clients.
* Adds a middleware to http clients to push the session_id into the
`X-Xet-Session-Id` header for outgoing requests (CAS is already
configured to accept this header).
* Adds info-level spans to the key parts of xet-core for cases where
xet-core is used as a library by long-running systems (e.g. migration
service or internal systems) for aid in debugging / tracing (essentially
bringing: #82 up to current
minus the hf_xet logging changes).
@jgodlew
Copy link
Contributor Author

jgodlew commented May 9, 2025

Closing since this was applied with #291

@jgodlew jgodlew closed this May 9, 2025
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.

2 participants