Skip to content

[ETW Exporter]Support serialize span/log attributes into JSON#1991

Merged
ThomsonTan merged 9 commits intoopen-telemetry:mainfrom
ThomsonTan:JasonSerializeETW
Feb 22, 2023
Merged

[ETW Exporter]Support serialize span/log attributes into JSON#1991
ThomsonTan merged 9 commits intoopen-telemetry:mainfrom
ThomsonTan:JasonSerializeETW

Conversation

@ThomsonTan
Copy link
Copy Markdown
Contributor

Fixes #1990

Changes

As ETW exporter is header only, this PR provide a C++ macro ENABLE_ENV_PROPERTIES, when defined, does serialization on all span/log attributes to a single attribute with JSON representation of all the attributes.

For significant contributions please make sure you have completed the following items:

  • CHANGELOG.md updated for non-trivial changes
  • Unit tests have been added
  • Changes in public API reviewed

@ThomsonTan ThomsonTan requested a review from a team February 18, 2023 00:34
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 18, 2023

Codecov Report

Merging #1991 (9d1b27e) into main (eed6572) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1991   +/-   ##
=======================================
  Coverage   87.29%   87.29%           
=======================================
  Files         166      166           
  Lines        4662     4662           
=======================================
  Hits         4069     4069           
  Misses        593      593           

Comment thread exporters/etw/include/opentelemetry/exporters/etw/etw_tracer.h
Comment thread exporters/etw/include/opentelemetry/exporters/etw/etw_tracer.h
Copy link
Copy Markdown
Member

@lalitb lalitb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Thanks for the fix.

@ThomsonTan ThomsonTan merged commit 5c121cf into open-telemetry:main Feb 22, 2023
@ThomsonTan ThomsonTan deleted the JasonSerializeETW branch February 22, 2023 04:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ETW Exporter] Support serializing span/log attributes to a single column in JSON

2 participants