Skip to content

[pdata] Replace Request.Set<Signals> with NewRequestFrom<Signals> #5234

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Apr 20, 2022

Conversation

dmitryax
Copy link
Member

  • plogotlp.Request.SetLogs func is deprecated in favor of plogotlp.NewRequestFromLogs
  • pmetricotlp.Request.SetMetrics func is deprecated in favor of pmetricotlp.NewRequestFromMetrics
  • ptraceotlp.Request.SetTraces func is deprecated in favor of ptraceotlp.NewRequestFromTraces

Closes #5230

@dmitryax dmitryax requested review from a team and jpkrohling April 20, 2022 00:05
@dmitryax dmitryax force-pushed the remove-request-set branch 2 times, most recently from ba44981 to 834fb87 Compare April 20, 2022 00:21
@codecov
Copy link

codecov bot commented Apr 20, 2022

Codecov Report

Merging #5234 (a871a60) into main (cdd11e7) will decrease coverage by 0.09%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #5234      +/-   ##
==========================================
- Coverage   90.65%   90.56%   -0.10%     
==========================================
  Files         187      187              
  Lines       11052    11052              
==========================================
- Hits        10019    10009      -10     
- Misses        814      823       +9     
- Partials      219      220       +1     
Impacted Files Coverage Δ
exporter/otlpexporter/otlp.go 90.42% <100.00%> (-0.30%) ⬇️
exporter/otlphttpexporter/otlp.go 81.73% <100.00%> (-0.47%) ⬇️
pdata/plog/plogotlp/logs.go 47.16% <100.00%> (-1.85%) ⬇️
pdata/pmetric/pmetricotlp/metrics.go 51.02% <100.00%> (-2.18%) ⬇️
pdata/ptrace/ptraceotlp/traces.go 47.16% <100.00%> (-1.85%) ⬇️
pdata/internal/common.go 94.16% <0.00%> (-0.78%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cdd11e7...a871a60. Read the comment docs.

- `plogotlp.Request.SetLogs` func is deprecated in favor of `plogotlp.NewRequestFromLogs`
- `pmetricotlp.Request.SetMetrics` func is deprecated in favor of `pmetricotlp.NewRequestFromMetrics`
- `ptraceotlp.Request.SetTraces` func is deprecated in favor of `ptraceotlp.NewRequestFromTraces`
@dmitryax dmitryax force-pushed the remove-request-set branch from 834fb87 to 6a7120f Compare April 20, 2022 01:06
Copy link
Member

@jpkrohling jpkrohling left a comment

Choose a reason for hiding this comment

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

I like the new API, and do prefer this over the MoveTo alternative.

dmitryax and others added 3 commits April 20, 2022 08:36
Co-authored-by: Juraci Paixão Kröhling <[email protected]>
Co-authored-by: Juraci Paixão Kröhling <[email protected]>
Co-authored-by: Juraci Paixão Kröhling <[email protected]>
@bogdandrutu bogdandrutu merged commit 87069a8 into open-telemetry:main Apr 20, 2022
@dmitryax dmitryax deleted the remove-request-set branch April 21, 2022 03:11
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.

Inconsistent Set[Traces|Metrics|Logs] on Request objects
4 participants