binarycat-dremio opened a new issue, #4494:
URL: https://github.com/apache/arrow-adbc/issues/4494
## Summary
The Go ADBC drivers currently use local deferred closures like `defer func()
{ internal.EndSpan(span, err) }()` so span completion sees the final named
`err` value.
## Problem
That pattern is correct, but easy to refactor incorrectly into `defer
internal.EndSpan(span, err)`, which would capture `err` too early.
## Follow-up
Introduce a small internal telemetry helper for defer-friendly span
lifecycle handling and migrate the current tracing call sites to it.
This is a follow-up refactor from #4487 rather than part of the tracing bug
fix itself.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
