-
Notifications
You must be signed in to change notification settings - Fork 60
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
Examples clarifications #17
Comments
Those are extremely valid points @marlon-sousa! |
Hello, I have yet another question that I think would be valid to be addressed. Because of diesel, I have to use a actix_web::web::block where I put the function calls that deal with database. If some of these functions cause a panic, at first nothing is captured by tracing subscriber. I then found a crate called tracing_unwrap, that is in fact sending the panic message to the subscriber, so that I have it logged nicely in JSON. Issue is, though, that the contextual information that I recorded in the root span is not being sent together, so that I have all the standard otel fields, but not the extra fields I have recorded in my root span. Flow is as follows:
It would be really good if I could have some understanding of how these spans are passed abroad, because they are not injected into the functions as parameters, so something else has to be happening. I have been reading a lot of documentation but it seems for me that every time I got a response it brings fifty more unanswered questions. Never thought the learning curve here would be so intense. Thanks, |
Hello,
Thanks for your work.
Being new to the whole eco system, I am still unsure how this library relates to others.
I like the idea of defining custom fields that can be filled at run time, but I am somewhat unsure about two questions I'd like to have clarified:
async fn personal_hello
I will try to dig around on this library, but again it is kind of a puzzle where it is very hard to understand who is responsible for what, including all the async corner cases. If you could add this kind of information on the example this would be great to new members!
Thanks,
Marlon
The text was updated successfully, but these errors were encountered: