-
Notifications
You must be signed in to change notification settings - Fork 59
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
Feature request: Make it so we can show spans that encapsulating handled exception as INFO level #142
Comments
Spans no longer have null levels. The backend defaults them to info.
There's a Personally I think the UI should have room to show both the current span level and some information about its descendant levels, e.g. an error bubble inside an info bubble or a small error bubble on the side. |
I think showing descendant information would be great! In the image contained in the original request, it would be awesome if the highest level one was info but still showed information that there were internally handled errors (like a bubble or something as you've mentioned). In the meantime, I'll check out the |
Since spans already default to info, it sounds like you shouldn't have a need for it. |
So the issue is that if I don't manually set anything and an error happens internally to the parent span, the parent span shows up as an error even though the error was actually handled internally (i.e. the call was retried and succeeded). I tried using Any idea what approach would be best here to get the desired behavior? I'm wondering if maybe there's a better solution on the UI side as you mentioned to somehow check if the error was handled before the span finished and then display that there was a handled error within the span? |
This is now resolved in the UI - the level of a span is distinguished from its descendants, and the bubble can have two colours. |
Description
In this case, the outermost task did a retry of a failure, and the retry succeeded.I think what we can do to better handle things today is, rather than showing the worst descendant level of a span in all circumstances, we do the following:
We probably also want to make sure there is an API for late-setting the level on a span so that, e.g. in the example above, we could leave the level unset until the request finally succeeds, then set the level to INFO right at the end when we know the inner call did succeed, as a way to get the bar to show up the desired color.
@alexmojaki if you agree with this, I am happy to handle the frontend side if you can help with the SDK. If you disagree with this approach, happy to discuss (I'm not 100% confident it doesn't have issues, but it made sense to me).
(Reported by @willbakst)
The text was updated successfully, but these errors were encountered: