Carlos Alberto Cortez

Results 203 comments of Carlos Alberto Cortez

Let's discuss it on today's call (with whoever can make it) - hopefully we can merge this (either with minor renaming or not) today ;)

@jmacd Do you have a strong instance against going with `process.threads`? This was briefly discussed in the past Spec call, and the Golang case was deemed as a very-specific case...

@lalitb This is a rather optional behavior, so it would be perfectly fine to not support it for C++ (although it's good you know about its existence, etc)

> I am very confused, and we are going to do very random decisions based on TC member personal's preference that approves that PR or does not look at that...

Is it to bad to suggest that _only_ root-level attributes _get_ a suffix (`short_name` becomes `scope.short_name` or `otel.scope.short_name`), as they are de facto considered meta attributes?

Approving, but as mentioned in the Spec call too, let's allow some days for some days to iron out final details.

> I would say we keep it as resource but add a note in the markdown that they may also be sent as span attributes. We may later want to...

Overall good, but we need more eyes on this. @open-telemetry/specs-approvers please take a look.

@bogdandrutu Any further opinion on this?