search

jackc / pgx

PostgreSQL driver and toolkit for Go
MIT License
10.84k stars 846 forks source link

Implement 'MultiTracer' #2084

Closed EpicStep closed 4 months ago

EpicStep commented 4 months ago

Hi. I'm tried to implement multi tracer (https://github.com/jackc/pgx/discussions/1677) I took the fact that in ConnConfig the main tracer is QueryTracer and I decided to split from it into different tracers inside the constructor. If the API is good, I'll write the tests.

jackc commented 4 months ago

Seems reasonable.

EpicStep commented 4 months ago

Should we support pgxpool Acquire/Release Tracer?

jackc commented 4 months ago

Should we support pgxpool Acquire/Release Tracer?

Yes.

EpicStep commented 4 months ago

I'm done. I moved multi tracer to a separate package to avoid an import cycle.