refactor: dont inherit sealed class

FantasticFiasco · Apr 2, 2024 · 9754860 · 9754860
1 parent ab1c866
commit 9754860
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 33 deletions.
diff --git a/src/Serilog.Sinks.Udp/LoggerSinkConfigurationExtensions.cs b/src/Serilog.Sinks.Udp/LoggerSinkConfigurationExtensions.cs
@@ -22,6 +22,7 @@
 using Serilog.Sinks.Udp.Private;
 using Serilog.Core;
 using Serilog.Debugging;
+using Serilog.Sinks.PeriodicBatching;
 
 namespace Serilog;
 
@@ -171,9 +172,16 @@ public static LoggerConfiguration Udp(
         try
         {
             var client = UdpClientFactory.Create(localPort, family, enableBroadcast);
-            var sink = new BatchingSink(new UdpSink(client, remoteAddress, remotePort, formatter, encoding));
+            var udpSink = new UdpSink(client, remoteAddress, remotePort, formatter, encoding);
+            var batchingSink = new PeriodicBatchingSink(
+                udpSink,
+                new PeriodicBatchingSinkOptions
+                {
+                    BatchSizeLimit = 1000,
+                    Period = TimeSpan.FromSeconds(0.5)
+                });
 
-            return sinkConfiguration.Sink(sink, restrictedToMinimumLevel, levelSwitch);
+            return sinkConfiguration.Sink(batchingSink, restrictedToMinimumLevel, levelSwitch);
         }
         catch (Exception e)
         {

diff --git a/src/Serilog.Sinks.Udp/Sinks/Udp/Private/BatchingSink.cs b/src/Serilog.Sinks.Udp/Sinks/Udp/Private/BatchingSink.cs