fixed a bug where multiple threads tried accessing the db context

cs-23-sw-7-06 · Dec 19, 2023 · b05e50e · b05e50e
1 parent 8d1420e
commit b05e50e
Show file tree

Hide file tree

Showing 5 changed files with 44 additions and 6 deletions.
diff --git a/CentralHub.Api.Tests/MockAggregatedMeasurementRepository.cs b/CentralHub.Api.Tests/MockAggregatedMeasurementRepository.cs
@@ -105,4 +105,17 @@ await RecentMeasurementGroupsMutex.Lock(recentMeasurementGroups =>
             return aggregatedMeasurements.Min(m => m.StartTime);
         }, cancellationToken);
     }
+
+    public async Task AddAggregatedMeasurementAsync(List<AggregatedMeasurementDto> measurementDtos, CancellationToken cancellationToken)
+    {
+        await AggregatedMeasurementsMutex.Lock(stuff =>
+        {
+            stuff.AggregatedMeasurements.AddRange(measurementDtos);
+            measurementDtos.ForEach(am =>
+            {
+                am.AggregatedMeasurementDtoId = stuff.NextId;
+                stuff.NextId++;
+            });
+        }, cancellationToken);
+    }
 }
diff --git a/CentralHub.Api/Program.cs b/CentralHub.Api/Program.cs
@@ -38,7 +38,7 @@ private static void Main(string[] args)
             // Ignore
         }
 #endif
-        var connectionString = $"Data Source={dbPath}";
+        var connectionString = $"Data Source={dbPath}; Pooling=False";
 
         builder.Services.AddDbContext<ApplicationDbContext>(options => options.UseSqlite(connectionString));
         builder.Services.AddScoped<IRoomRepository, RoomRepository>();

diff --git a/CentralHub.Api/Services/IMeasurementRepository.cs b/CentralHub.Api/Services/IMeasurementRepository.cs
@@ -6,6 +6,8 @@ namespace CentralHub.Api.Services;
 
 public interface IMeasurementRepository
 {
+    Task AddAggregatedMeasurementAsync(List<AggregatedMeasurementDto> measurementDtos, CancellationToken cancellationToken);
+
     Task<int> AddAggregatedMeasurementAsync(AggregatedMeasurementDto measurementDto, CancellationToken cancellationToken);
 
     Task RemoveAggregatedMeasurementAsync(AggregatedMeasurementDto measurementDto, CancellationToken cancellationToken);

diff --git a/CentralHub.Api/Services/LocalizationService.cs b/CentralHub.Api/Services/LocalizationService.cs
@@ -3,6 +3,7 @@
 using CentralHub.Api.Model;
 using CentralHub.Api.Model.Responses.Measurement;
 using CentralHub.Api.Threading;
+using Microsoft.Data.Sqlite;
 
 namespace CentralHub.Api.Services;
 
@@ -44,6 +45,8 @@ public async Task AggregateMeasurementsAsync(CancellationToken stoppingToken)
         var roomMeasurementGroups = await aggregatorRepository.GetRoomMeasurementGroupsAsync(stoppingToken);
         var allRooms = await roomRepository.GetRoomsAsync(stoppingToken);
         using var measuredRoomsMutex = new CancellableMutex<List<RoomDto>>(new List<RoomDto>());
+        using var aggregatedMeasurementsMutex = new CancellableMutex<List<AggregatedMeasurementDto>>(new List<AggregatedMeasurementDto>());
+
 
         await Parallel.ForEachAsync(
             roomMeasurementGroups,
@@ -60,21 +63,25 @@ await Parallel.ForEachAsync(
 
             await measuredRoomsMutex.Lock(m => m.Add(room), cancellationToken);
 
-            await aggregatorRepository.AddAggregatedMeasurementAsync(
-                CreateAggregatedMeasurement(room, roomMeasurementGroup.Value),
-                cancellationToken);
+            var aggregatedMeasurement = CreateAggregatedMeasurement(room, roomMeasurementGroup.Value);
+            await aggregatedMeasurementsMutex.Lock(am => am.Add(aggregatedMeasurement), cancellationToken);
         });
 
         await measuredRoomsMutex.Lock(async m =>
         {
             foreach (var room in allRooms.Where(r => !m.Contains(r)))
             {
-                await aggregatorRepository.AddAggregatedMeasurementAsync(
-                    CreateAggregatedMeasurement(room, new List<MeasurementGroup>()),
+                await aggregatedMeasurementsMutex.Lock(am =>
+                    am.Add(CreateAggregatedMeasurement(room, new List<MeasurementGroup>())),
                     stoppingToken);
+
             }
         },
         stoppingToken);
+
+        await aggregatedMeasurementsMutex.Lock(async am =>
+            await aggregatorRepository.AddAggregatedMeasurementAsync(am, stoppingToken),
+            stoppingToken);
     }
 
     private static AggregatedMeasurementDto CreateAggregatedMeasurement(RoomDto room, IReadOnlyCollection<MeasurementGroup> measurementGroups)

diff --git a/CentralHub.Api/Services/MeasurementRepository.cs b/CentralHub.Api/Services/MeasurementRepository.cs
@@ -22,6 +22,22 @@ public MeasurementRepository(ApplicationDbContext applicationDbContext)
         _applicationDbContext.Database.EnsureCreated();
     }
 
+    public async Task AddAggregatedMeasurementAsync(List<AggregatedMeasurementDto> measurementDtos, CancellationToken cancellationToken)
+    {
+        _applicationDbContext.AggregatedMeasurements.AddRange(measurementDtos);
+        try
+        {
+            await _applicationDbContext.SaveChangesAsync(cancellationToken);
+            return;
+        }
+        catch (OperationCanceledException)
+        {
+            // Remove the roomDto from the collection as the operation was cancelled.
+            _applicationDbContext.AggregatedMeasurements.RemoveRange(measurementDtos);
+            throw;
+        }
+    }
+
     public async Task<int> AddAggregatedMeasurementAsync(AggregatedMeasurementDto measurementDto, CancellationToken cancellationToken)
     {
         _applicationDbContext.AggregatedMeasurements.Add(measurementDto);