semantic-conventions icon indicating copy to clipboard operation
semantic-conventions copied to clipboard

dynamodb attributes are in aws namespace instead db

Open Flarna opened this issue 2 years ago • 6 comments
trafficstars

The span attributes for dynamodb are currently in aws namespace.

Other databases like mondodb or mssql have their special attributes in the db namespace.

At other places like cloud.region vendor namespaces are also replaced/avoided by a generic namespace.

Should the dynamodb attributes moved to db namespace?

Flarna avatar Jul 10 '23 17:07 Flarna

I think it's kind of a wash. All aws-API-specific attributes start with aws., all db-related ones with db.. You can't have both. There are some remarks regarding use of db.* e.g. at https://github.com/open-telemetry/opentelemetry-specification/pull/1422#issuecomment-789672353

Oberon00 avatar Jul 11 '23 08:07 Oberon00

This was closed by mistake by the stale bot. Re-opening

joaopgrassi avatar Feb 26 '24 09:02 joaopgrassi

fyi @trask

joaopgrassi avatar Feb 26 '24 09:02 joaopgrassi

I'd vote for the db.* prefix to be given precedence since that seems like the more important grouping from semantic convention perspective.

trask avatar Apr 24 '24 19:04 trask

even if we aren't targeting dynamodb as part of initial database semconv stability, I think we should make this change as part of the initial database semconv stability process so that this breaking change could benefit from being grouped with the other breaking changes related to stabilization

trask avatar Apr 26 '24 20:04 trask

I thought about this some more and changed my opinion.

we should try to stick with the philosophy "try to consolidate all breaking changes into a single bump when going stable"

and so we should defer this change until we are planing dynamodb semconv stabilization (which is not included as part of initial db semconv stability)

trask avatar May 08 '24 22:05 trask

Closing as we are now intentionally capturing aws-specific attributes under aws.* namespace

trask avatar May 09 '25 15:05 trask