-
Notifications
You must be signed in to change notification settings - Fork 189
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
What's the use case for exception.escaped
?
#1516
Comments
The trace sdk in python seems to be setting this attribute to
The documentation for recording an exception has a pseudo-java example - maybe we should replace it with the python example? The Java example still demonstrates the purpose of this attribute. |
thanks @scheler I think the question I'm trying to ask is what would a user do with this piece of information ( |
@trask I think the python community would know more on this since the spec language for this attribute includes python terms and is likely written by someone from python side. (ContextManager / exit method). |
I was making a case for this attribute in open-telemetry/opentelemetry-specification#4333, but I no longer think it's necessary. I believe the use-case behind it is the ability to distinguish handled exceptions from unhandled ones. If we report exceptions on logs/events, we have severity to capture it - there is no severity on the span event and we should deprecated this attribute along with span events. |
The attribute is now deprecated in #1716 |
What's the use case for
exception.escaped
?We have never captured this in Java instrumentation, and no one has ever asked for it, so I'm wondering if it's really useful?
Assuming there are use cases for it, are they compelling enough to justify capturing it by default (as recommended) or should it be opt-in?
The text was updated successfully, but these errors were encountered: