Skip to content

Refactor ConsoleExporter#1593

Merged
CodeBlanch merged 13 commits into
open-telemetry:masterfrom
utpilla:utpilla/Refactor-ConsoleExporter
Nov 25, 2020
Merged

Refactor ConsoleExporter#1593
CodeBlanch merged 13 commits into
open-telemetry:masterfrom
utpilla:utpilla/Refactor-ConsoleExporter

Conversation

@utpilla

@utpilla utpilla commented Nov 19, 2020

Copy link
Copy Markdown
Contributor

Changes

  • Refactored ConsoleExporter to get rid of type specific check in the generic class ConsoleExporter
  • Changed ConsoleExporter to abstract
  • Added ConsoleActivityExporter and ConsoleLogRecordExported

@utpilla utpilla requested a review from a team November 19, 2020 21:54
@codecov

codecov Bot commented Nov 19, 2020

Copy link
Copy Markdown

Codecov Report

Merging #1593 (c4f1332) into master (4ccdb6e) will increase coverage by 0.11%.
The diff coverage is 5.45%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1593      +/-   ##
==========================================
+ Coverage   80.56%   80.68%   +0.11%     
==========================================
  Files         243      245       +2     
  Lines        6582     6575       -7     
==========================================
+ Hits         5303     5305       +2     
+ Misses       1279     1270       -9     
Impacted Files Coverage Δ
...xporter.Console/ConsoleExporterHelperExtensions.cs 0.00% <0.00%> (ø)
...porter.Console/ConsoleExporterLoggingExtensions.cs 0.00% <0.00%> (ø)
...metry.Exporter.Console/ConsoleLogRecordExporter.cs 0.00% <0.00%> (ø)
...emetry.Exporter.Console/ConsoleActivityExporter.cs 5.71% <5.71%> (ø)
.../OpenTelemetry.Exporter.Console/ConsoleExporter.cs 42.85% <50.00%> (+38.24%) ⬆️

@cijothomas cijothomas left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Please add this to changelog as there is public class changes.

Comment thread src/OpenTelemetry.Exporter.Console/ConsoleExporter.cs Outdated
Comment thread src/OpenTelemetry.Exporter.Console/ConsoleExporter.cs
Comment thread src/OpenTelemetry.Exporter.Console/ConsoleExporter.cs Outdated
Comment thread src/OpenTelemetry.Exporter.Console/ConsoleLogRecordExporter.cs
Comment thread src/OpenTelemetry.Exporter.Console/ConsoleActivityExporter.cs
@cijothomas

Copy link
Copy Markdown
Member

Okay to merge, with one more approval.

@CodeBlanch CodeBlanch merged commit 7deee54 into open-telemetry:master Nov 25, 2020
@utpilla utpilla deleted the utpilla/Refactor-ConsoleExporter branch November 30, 2020 22:25
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.

5 participants