Mark internal classes sealed (#37724)

* Mark internal classes sealed * exporters
Azure · Jul 19, 2023 · 84803ca · 84803ca
1 parent c940870
commit 84803ca
Show file tree

Hide file tree

Showing 13 changed files with 13 additions and 13 deletions.
diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/AzureMonitorLogExporter.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/AzureMonitorLogExporter.cs
@@ -11,7 +11,7 @@
 
 namespace Azure.Monitor.OpenTelemetry.Exporter
 {
-    internal class AzureMonitorLogExporter : BaseExporter<LogRecord>
+    internal sealed class AzureMonitorLogExporter : BaseExporter<LogRecord>
     {
         private readonly ITransmitter _transmitter;
         private readonly string _instrumentationKey;

diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/AzureMonitorMetricExporter.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/AzureMonitorMetricExporter.cs
@@ -11,7 +11,7 @@
 
 namespace Azure.Monitor.OpenTelemetry.Exporter
 {
-    internal class AzureMonitorMetricExporter : BaseExporter<Metric>
+    internal sealed class AzureMonitorMetricExporter : BaseExporter<Metric>
     {
         private readonly ITransmitter _transmitter;
         private readonly string _instrumentationKey;

diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/AzureMonitorTraceExporter.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/AzureMonitorTraceExporter.cs
@@ -11,7 +11,7 @@
 
 namespace Azure.Monitor.OpenTelemetry.Exporter
 {
-    internal class AzureMonitorTraceExporter : BaseExporter<Activity>
+    internal sealed class AzureMonitorTraceExporter : BaseExporter<Activity>
     {
         private readonly ITransmitter _transmitter;
         private readonly string _instrumentationKey;

diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/AksResourceProcessor.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/AksResourceProcessor.cs
@@ -6,7 +6,7 @@
 
 namespace Azure.Monitor.OpenTelemetry.Exporter.Internals;
 
-internal class AksResourceProcessor
+internal sealed class AksResourceProcessor
 {
     private static readonly Dictionary<string, Action<AksResourceProcessor, string>> s_propertySetters = new(7)
     {

diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/ApplicationInsightsSampler.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/ApplicationInsightsSampler.cs
@@ -11,7 +11,7 @@ namespace Azure.Monitor.OpenTelemetry.Exporter.Internals;
 /// Sample configurable for OpenTelemetry exporters for compatibility
 /// with Application Insight SDKs.
 /// </summary>
-internal class ApplicationInsightsSampler : Sampler
+internal sealed class ApplicationInsightsSampler : Sampler
 {
     private static readonly SamplingResult RecordOnlySamplingResult = new(SamplingDecision.RecordOnly);
     private readonly SamplingResult recordAndSampleSamplingResult;

diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/AzureMonitorResource.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/AzureMonitorResource.cs
@@ -5,7 +5,7 @@
 
 namespace Azure.Monitor.OpenTelemetry.Exporter.Internals
 {
-    internal class AzureMonitorResource
+    internal sealed class AzureMonitorResource
     {
         internal string? RoleName { get; set; }
 

diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/LogsHelper.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/LogsHelper.cs
@@ -18,7 +18,7 @@
 
 namespace Azure.Monitor.OpenTelemetry.Exporter.Internals
 {
-    internal class LogsHelper
+    internal static class LogsHelper
     {
         private const int Version = 2;
         private static readonly ConcurrentDictionary<int, string> s_depthCache = new ConcurrentDictionary<int, string>();

diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/MetricHelper.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/MetricHelper.cs
@@ -11,7 +11,7 @@
 
 namespace Azure.Monitor.OpenTelemetry.Exporter.Internals
 {
-    internal class MetricHelper
+    internal static class MetricHelper
     {
         private const int Version = 2;
 

diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/NDJsonWriter.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/NDJsonWriter.cs
@@ -7,7 +7,7 @@
 
 namespace Azure.Monitor.OpenTelemetry.Exporter.Internals
 {
-    internal class NDJsonWriter : IDisposable
+    internal sealed class NDJsonWriter : IDisposable
     {
         private static readonly byte[] s_separator = { (byte)'\n' };
 

diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/ResponseStatusCodes.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/ResponseStatusCodes.cs
@@ -3,7 +3,7 @@
 
 namespace Azure.Monitor.OpenTelemetry.Exporter.Internals
 {
-    internal class ResponseStatusCodes
+    internal static class ResponseStatusCodes
     {
         public const int Success = 200;
         public const int PartialSuccess = 206;

diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/StandardMetricConstants.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/StandardMetricConstants.cs
@@ -3,7 +3,7 @@
 
 namespace Azure.Monitor.OpenTelemetry.Exporter.Internals
 {
-    internal class StandardMetricConstants
+    internal static class StandardMetricConstants
     {
         internal const string StandardMetricMeterName = "StandardMetricMeter";
         internal const string RequestDurationInstrumentName = "RequestDurationStandardMetric";

diff --git a/.../Azure.Monitor.OpenTelemetry.Exporter/src/Internals/StandardMetricsExtractionProcessor.cs b/.../Azure.Monitor.OpenTelemetry.Exporter/src/Internals/StandardMetricsExtractionProcessor.cs
@@ -11,7 +11,7 @@
 
 namespace Azure.Monitor.OpenTelemetry.Exporter.Internals
 {
-    internal class StandardMetricsExtractionProcessor : BaseProcessor<Activity>
+    internal sealed class StandardMetricsExtractionProcessor : BaseProcessor<Activity>
     {
         private bool _disposed;
         private AzureMonitorResource? _resource;

diff --git a/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/TransmitterFactory.cs b/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/TransmitterFactory.cs
@@ -12,7 +12,7 @@ namespace Azure.Monitor.OpenTelemetry.Exporter.Internals
     /// This factory should ensure that only one instance of the Transmitter is created for
     /// any unique connection string.
     /// </summary>
-    internal class TransmitterFactory
+    internal sealed class TransmitterFactory
     {
         public static readonly TransmitterFactory Instance = new();
         public static readonly IPlatform platform = new DefaultPlatform();