Investigate a RecordException like method for an API Span

[MrAlias]

The specification has added a RecordException method to the API Span. This is to record things like a Go panic as a Span event.

We currently only have a RecordError method on the Span. This method is not something specified in the specification and doesn't seem suited to record a runtime.Stack.

Should the RecordError method be removed?

Should we add a method that helps in panic recovery? Possibly something that can be deferred and adds an event if a panic is occuring.

[Aneurysm9]

I think we might want to add an option to RecordError to trigger recording of a stack trace, perhaps adding a RecordException convenience wrapper that automatically adds that option to a RecordError invocation.

As for panic recovery, I think the existing RecordError or proposed RecordException would be good candidates for users who are trapping their own panics. The span.End() method has a panic recovery mechanism built in that adds an event with the error information. Adding a stack trace as attributes to that event would probably be a good idea. Perhaps it could wrap the received error before re-panicing with a sentinel error type to avoid recording the same error on an entire stack of spans that are being unwound.

[MrAlias]

This can likely be closed. We need to double check, but this is likely in line with the specification.

[MrAlias]

Consolidating the conversation in #1491