Skip to content

Commit

Permalink
[ETW Exporter]Do not overwrite ParentId when setting attribute on Span (
Browse files Browse the repository at this point in the history
  • Loading branch information
ThomsonTan authored Feb 18, 2023
1 parent 01eac4c commit 95f6c66
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
2 changes: 2 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,8 @@ Increment the:

## [Unreleased]

* ETW Exporter]Do not overwrite ParentId when setting attribute on Span
[#1989](https://github.com/open-telemetry/opentelemetry-cpp/pull/1989)
* Convert Prometheus Exporter to Pull MetricReader [#1953](https://github.com/open-telemetry/opentelemetry-cpp/pull/1953)
* Upgrade prometheus-cpp to v1.1.0 [#1954](https://github.com/open-telemetry/opentelemetry-cpp/pull/1954)
* [BUILD] Build OpenTelemetry SDK and exporters into DLL
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -870,7 +870,8 @@ class Span : public opentelemetry::trace::Span
void SetAttribute(nostd::string_view key, const common::AttributeValue &value) noexcept override
{
// don't override fields propagated from span data.
if (key == ETW_FIELD_NAME || key == ETW_FIELD_SPAN_ID || key == ETW_FIELD_TRACE_ID)
if (key == ETW_FIELD_NAME || key == ETW_FIELD_SPAN_ID || key == ETW_FIELD_TRACE_ID ||
key == ETW_FIELD_SPAN_PARENTID)
{
return;
}
Expand Down

0 comments on commit 95f6c66

Please sign in to comment.