Skip to content

Commit

Permalink
Refactor exporter - step 8 (#1127)
Browse files Browse the repository at this point in the history
* remove SimpleActivityProcessor

* update changelog

* remove ActivityExporter
  • Loading branch information
reyang authored Aug 21, 2020
1 parent ee5b038 commit 9eccaf4
Show file tree
Hide file tree
Showing 7 changed files with 11 additions and 490 deletions.
9 changes: 9 additions & 0 deletions src/OpenTelemetry/CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,15 @@
[#1035](https://github.com/open-telemetry/opentelemetry-dotnet/pull/1035))
* Changed `AddActivitySource` to `AddSource` with params support
([#1036](https://github.com/open-telemetry/opentelemetry-dotnet/pull/1036))
* Refactored simple export and batch export APIs
([#1078](https://github.com/open-telemetry/opentelemetry-dotnet/pull/1078)
[#1081](https://github.com/open-telemetry/opentelemetry-dotnet/pull/1081)
[#1083](https://github.com/open-telemetry/opentelemetry-dotnet/pull/1083)
[#1085](https://github.com/open-telemetry/opentelemetry-dotnet/pull/1085)
[#1087](https://github.com/open-telemetry/opentelemetry-dotnet/pull/1087)
[#1094](https://github.com/open-telemetry/opentelemetry-dotnet/pull/1094)
[#1113](https://github.com/open-telemetry/opentelemetry-dotnet/pull/1113)
[#1127](https://github.com/open-telemetry/opentelemetry-dotnet/pull/1127))

## 0.4.0-beta.2

Expand Down
2 changes: 1 addition & 1 deletion src/OpenTelemetry/Internal/OpenTelemetrySdkEventSource.cs
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ public void ShutdownEvent(int spansLeftUnprocessed)
}

[Event(3, Message = "Exporter returned error '{0}'.", Level = EventLevel.Warning)]
public void ExporterErrorResult(ExportResult exportResult)
public void ExporterErrorResult(ExportResultSync exportResult)
{
this.WriteEvent(3, exportResult.ToString());
}
Expand Down
80 changes: 0 additions & 80 deletions src/OpenTelemetry/Trace/ActivityExporter.cs

This file was deleted.

99 changes: 0 additions & 99 deletions src/OpenTelemetry/Trace/SimpleActivityProcessor.cs

This file was deleted.

72 changes: 0 additions & 72 deletions test/OpenTelemetry.Tests/Shared/TestActivityExporter.cs

This file was deleted.

Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ public class CompositeActivityProcessorTests
public void CompositeActivityProcessor_BadArgs()
{
Assert.Throws<ArgumentNullException>(() => new CompositeActivityProcessor(null));
Assert.Throws<ArgumentException>(() => new CompositeActivityProcessor(new SimpleActivityProcessor[0]));
Assert.Throws<ArgumentException>(() => new CompositeActivityProcessor(new ActivityProcessor[0]));
}

[Fact]
Expand Down
Loading

0 comments on commit 9eccaf4

Please sign in to comment.