An error occurred while fetching this tab.
Replace internal tracing hooks with otelsql
There are no commits yet
Push commits to the source branch or add previously merged commits to review them.
Created by: jhchabran
Replace our in-house tracing for sql integration so it uses otelsql
instead. We're getting more details and tracing on the driver level.
Notes:
main()
functions.
db.args.$n: value
.sqlhooks
probably needs to be patched, as it's missing one of the few optional methods and using it with otelsql.Open
yields some issues. That being said it's highly probable that we end up not needing it at all because otelsql would handle everything.
Tested locally, see https://github.com/sourcegraph/sourcegraph/issues/37905#issuecomment-1192557027
Update: added the arguments as attributes:
Push commits to the source branch or add previously merged commits to review them.