Use EventIdPropertyCache in SerilogLogger; Create EventIdPropertyCache test class.

Author: andrii-babanin-sym

src/Serilog.Extensions.Logging/Extensions/Logging/SerilogLogger.cs

@@ -29,12 +29,12 @@ internal static string GetKeyWithoutFirstSymbol(ConcurrentDictionary<string, str
     readonly SerilogLoggerProvider _provider;
     readonly ILogger _logger;
 
-    static readonly CachingMessageTemplateParser MessageTemplateParser = new();
+    /// Per-ILogger LogEventProperty cache for EventId.
+    /// Each ILogger has its own cache.
+    /// Each { ILogger, EventId.Id } pair is assumed to be unique.
+    readonly EventIdPropertyCache _eventPropertyCache = new (48);
 
-    // It's rare to see large event ids, as they are category-specific
-    static readonly LogEventProperty[] LowEventIdValues = Enumerable.Range(0, 48)
-        .Select(n => new LogEventProperty("Id", new ScalarValue(n)))
-        .ToArray();
+    static readonly CachingMessageTemplateParser MessageTemplateParser = new();
 
     public SerilogLogger(
         SerilogLoggerProvider provider,
@@ -155,7 +155,7 @@ LogEvent PrepareWrite<TState>(LogEventLevel level, EventId eventId, TState state
         }
 
         if (eventId.Id != 0 || eventId.Name != null)
-            properties.Add(CreateEventIdProperty(eventId));
+            properties.Add(this._eventPropertyCache.GetOrCreateProperty(in eventId));
 
         var (traceId, spanId) = Activity.Current is { } activity ?
             (activity.TraceId, activity.SpanId) :
@@ -172,25 +172,4 @@ LogEvent PrepareWrite<TState>(LogEventLevel level, EventId eventId, TState state
             stateObj = formatter(state, null);
         return stateObj ?? state;
     }
-
-    internal static LogEventProperty CreateEventIdProperty(EventId eventId)
-    {
-        var properties = new List<LogEventProperty>(2);
-
-        if (eventId.Id != 0)
-        {
-            if (eventId.Id >= 0 && eventId.Id < LowEventIdValues.Length)
-                // Avoid some allocations
-                properties.Add(LowEventIdValues[eventId.Id]);
-            else
-                properties.Add(new LogEventProperty("Id", new ScalarValue(eventId.Id)));
-        }
-
-        if (eventId.Name != null)
-        {
-            properties.Add(new LogEventProperty("Name", new ScalarValue(eventId.Name)));
-        }
-
-        return new LogEventProperty("EventId", new StructureValue(properties));
-    }
 }

test/Serilog.Extensions.Logging.Tests/EventIdPropertyCacheTests.cs

@@ -0,0 +1,36 @@
+// Copyright (c) .NET Foundation. All rights reserved.
+// Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
+
+using Microsoft.Extensions.Logging;
+using Serilog.Events;
+using Xunit;
+
+namespace Serilog.Extensions.Logging.Tests
+{
+    public class EventIdPropertyCacheTests
+    {
+        [Theory]
+        [InlineData(1)]
+        [InlineData(10)]
+        [InlineData(48)]
+        [InlineData(100)]
+        public void LowAndHighNumberedEventIdsAreMapped(int id)
+        {
+            // Arrange
+            var cache = new EventIdPropertyCache(48);
+
+            var eventId = new EventId(id, "test");
+
+            // Act
+            var mapped = cache.GetOrCreateProperty(eventId);
+
+            // Assert
+            var value = Assert.IsType<StructureValue>(mapped.Value);
+            Assert.Equal(2, value.Properties.Count);
+
+            var idValue = value.Properties.Single(p => p.Name == "Id").Value;
+            var scalar = Assert.IsType<ScalarValue>(idValue);
+            Assert.Equal(id, scalar.Value);
+        }
+    }
+}

test/Serilog.Extensions.Logging.Tests/SerilogLoggerTests.cs

@@ -517,22 +517,6 @@ public void Dispose()
         }
     }
 
-    [Theory]
-    [InlineData(1)]
-    [InlineData(10)]
-    [InlineData(48)]
-    [InlineData(100)]
-    public void LowAndHighNumberedEventIdsAreMapped(int id)
-    {
-        var orig = new EventId(id, "test");
-        var mapped = SerilogLogger.CreateEventIdProperty(orig);
-        var value = Assert.IsType<StructureValue>(mapped.Value);
-        Assert.Equal(2, value.Properties.Count);
-        var idValue = value.Properties.Single(p => p.Name == "Id").Value;
-        var scalar = Assert.IsType<ScalarValue>(idValue);
-        Assert.Equal(id, scalar.Value);
-    }
-
     [Fact]
     public void MismatchedMessageTemplateParameterCountIsHandled()
     {