Skip to content

Support links and event attributes in Recordable#197

Merged
reyang merged 7 commits into
open-telemetry:masterfrom
nadiaciobanu:recordable-update
Jul 22, 2020
Merged

Support links and event attributes in Recordable#197
reyang merged 7 commits into
open-telemetry:masterfrom
nadiaciobanu:recordable-update

Conversation

@nadiaciobanu
Copy link
Copy Markdown
Contributor

Update the Recordable base class to support links and attributes in events. The implementation of these will be handled in further PRs.

@nadiaciobanu nadiaciobanu requested a review from a team July 21, 2020 19:15
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 21, 2020

Codecov Report

Merging #197 into master will decrease coverage by 0.03%.
The diff coverage is 50.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #197      +/-   ##
==========================================
- Coverage   95.72%   95.69%   -0.04%     
==========================================
  Files          96       96              
  Lines        2877     2878       +1     
==========================================
  Hits         2754     2754              
- Misses        123      124       +1     
Impacted Files Coverage Δ
sdk/include/opentelemetry/sdk/trace/recordable.h 100.00% <ø> (ø)
sdk/test/trace/span_data_test.cc 100.00% <ø> (ø)
sdk/include/opentelemetry/sdk/trace/span_data.h 98.33% <50.00%> (-1.67%) ⬇️

Comment thread exporters/otlp/include/opentelemetry/exporters/otlp/recordable.h Outdated
Copy link
Copy Markdown
Member

@reyang reyang left a comment

Choose a reason for hiding this comment

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

LGTM.

Comment thread exporters/otlp/include/opentelemetry/exporters/otlp/recordable.h Outdated
Comment thread exporters/otlp/include/opentelemetry/exporters/otlp/recordable.h Outdated
Comment thread exporters/otlp/include/opentelemetry/exporters/otlp/recordable.h
@reyang reyang added the pr:please-merge This PR is ready to be merged by a Maintainer (rebased, CI passed, has enough valid approvals, etc.) label Jul 22, 2020
@reyang reyang merged commit eb7cbd1 into open-telemetry:master Jul 22, 2020
@nadiaciobanu nadiaciobanu changed the title Support links and event attributes in recordable Support links and event attributes in Recordable Jul 23, 2020
@nadiaciobanu nadiaciobanu deleted the recordable-update branch July 23, 2020 22:35
GerHobbelt pushed a commit to GerHobbelt/opentelemetry-cpp that referenced this pull request Jun 17, 2025
…el_lib-2.x

Update dependency aspect_bazel_lib to v2.14.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr:please-merge This PR is ready to be merged by a Maintainer (rebased, CI passed, has enough valid approvals, etc.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants