instead of looking up all snapshot ids when there's more than 10 comm…

…its, lookup just the ids for entities changed
sillsdev · Oct 7, 2024 · a767219 · a767219
1 parent 39204db
commit a767219
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/src/SIL.Harmony/DataModel.cs b/src/SIL.Harmony/DataModel.cs
@@ -151,10 +151,13 @@ private async Task UpdateSnapshots(Commit oldestAddedCommit, Commit[] newCommits
     {
         await _crdtRepository.DeleteStaleSnapshots(oldestAddedCommit);
         Dictionary<Guid, Guid?> snapshotLookup;
-        //this is a performance optimization to avoid loading all the snapshots, this number is somewhat arbitrary
         if (newCommits.Length > 10)
         {
-            snapshotLookup = await _crdtRepository.CurrentSnapshots().ToDictionaryAsync(s => s.EntityId, s => (Guid?) s.Id);
+            var entityIds = newCommits.SelectMany(c => c.ChangeEntities.Select(ce => ce.EntityId));
+            snapshotLookup = await _crdtRepository.CurrentSnapshots()
+                .Where(s => entityIds.Contains(s.EntityId))
+                .Select(s => new KeyValuePair<Guid, Guid?>(s.EntityId, s.Id))
+                .ToDictionaryAsync(s => s.Key, s => s.Value);
         }
         else
         {