From 053ee69fea66d7bee5d7c058867e37812ca2b1f9 Mon Sep 17 00:00:00 2001 From: David Fowler Date: Sat, 28 Mar 2020 19:45:26 -0700 Subject: [PATCH] Lock around adding cached entries. (#243) * Use a ConcurrentQueue Fixed #179 --- src/Microsoft.Tye.Hosting/Model/Service.cs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Microsoft.Tye.Hosting/Model/Service.cs b/src/Microsoft.Tye.Hosting/Model/Service.cs index 58e7cf0d..03862258 100644 --- a/src/Microsoft.Tye.Hosting/Model/Service.cs +++ b/src/Microsoft.Tye.Hosting/Model/Service.cs @@ -19,7 +19,7 @@ namespace Microsoft.Tye.Hosting.Model { if (CachedLogs.Count > 5000) { - CachedLogs.Dequeue(); + CachedLogs.TryDequeue(out _); } CachedLogs.Enqueue(entry); @@ -64,7 +64,7 @@ namespace Microsoft.Tye.Hosting.Model public Dictionary Items { get; } = new Dictionary(); - public Queue CachedLogs { get; } = new Queue(); + public ConcurrentQueue CachedLogs { get; } = new ConcurrentQueue(); public Subject Logs { get; } = new Subject();