pgx query times report as application times in the Datadog UI about dd-trace-go HOT 3 OPEN

@rarguelloF @zarirhamza Can you review this? Thanks!

from dd-trace-go.

@ewhauser Sorry for the delayed response.

From the screenshot you are sending, it seems the postgres.db service disappeared, which is something expected as the pgx.v5 integration by default inherits the service name from DD_SERVICE instead of setting a custom one. We are aware this is an inconsistent behavior between different instrumentations, but overall the preferred behavior from now on is to get rid of these "fake" services (like postgres.db) and instead using DD_SERVICE everywhere in the same service. The DD_TRACE_REMOVE_INTEGRATION_SERVICE_NAMES_ENABLED environment variable ensures this behavior is applied in every integration/instrumentation you are using.

If you want to have the old behavior (having your pgx.v5 spans with a different service name), you can do the following:

import (
	pgxtrace "gopkg.in/DataDog/dd-trace-go.v1/contrib/jackc/pgx.v5"
)

// ...

db, err := pgxtrace.Connect(ctx, dbUrl, pgxtrace.WithServiceName("postgres.db"))

from dd-trace-go.