log: add error field to Record and make SDK to emit exception attributes

[iblancasa]

Fixes #7923

$ go test -run=^$ -bench=BenchmarkLoggerEmitExceptionAttributes -benchmem -count=5 -benchtime=500ms -cpu=1
goos: darwin
goarch: arm64
pkg: go.opentelemetry.io/otel/sdk/log
cpu: Apple M4 Pro
BenchmarkLoggerSetErrAndEmit                 	  628162	      1023 ns/op	    5371 B/op	       1 allocs/op
BenchmarkLoggerSetErrAndEmit                 	  663955	       863.3 ns/op	    5105 B/op	       1 allocs/op
BenchmarkLoggerSetErrAndEmit                 	  653888	      1067 ns/op	    5177 B/op	       1 allocs/op
BenchmarkLoggerSetErrAndEmit                 	  716438	       824.8 ns/op	    4764 B/op	       1 allocs/op
BenchmarkLoggerSetErrAndEmit                 	  746902	       999.2 ns/op	    5630 B/op	       1 allocs/op
BenchmarkLoggerSetExceptionAttributesAndEmit 	  650696	      1042 ns/op	    5200 B/op	       1 allocs/op
BenchmarkLoggerSetExceptionAttributesAndEmit 	  574962	       980.7 ns/op	    4743 B/op	       1 allocs/op
BenchmarkLoggerSetExceptionAttributesAndEmit 	  536736	       989.2 ns/op	    5049 B/op	       1 allocs/op
BenchmarkLoggerSetExceptionAttributesAndEmit 	  558511	      1190 ns/op	    4870 B/op	       1 allocs/op
BenchmarkLoggerSetExceptionAttributesAndEmit 	  669452	       978.8 ns/op	    5067 B/op	       1 allocs/op
PASS
ok  	go.opentelemetry.io/otel/sdk/log	6.994s

TODO after merged:

• log: add error field to Record and make SDK to emit exception attributes #7924 (comment)

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.6%. Comparing base (fdd1320) to head (245fa6e).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #7924   +/-   ##
=====================================
  Coverage   81.6%   81.6%           
=====================================
  Files        304     304           
  Lines      23389   23424   +35     
=====================================
+ Hits       19094   19132   +38     
+ Misses      3907    3904    -3     
  Partials     388     388           

Files with missing lines	Coverage Δ
log/record.go	100.0% <100.0%> (ø)
sdk/log/logger.go	100.0% <100.0%> (ø)

... and 2 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[pellared]

Please address the comments and add some benchmarks that would showcase and so that we can compare the difference between adding the attributes manually vs via the newly added API.

most comments addressed

[pellared]

I made a quick look and overall it looks acceptable.
Please add the benchmark results to the PR description.
Moreover, I would prefer handling the stack trace in a seperate PR and track this on a separate issue.
Thanks a lot for your work.

[pellared]

#7924 (comment) this comment is not addressed and the linter is failing.

Side note: Overall the PR looks nice 😉 💪

[iblancasa]

#7924 (comment) this comment is not addressed

Sorry I thought with the introduced changes was ok. I did some modifications and I think it should be correct now. Please, let me know if you want some other change or I'm missing something else.

and the linter is failing

I was not sure about fixing this or not because #7924 (comment) Will fix in a better way.

[pellared]

Thanks! 🏅