notfire/Iceshrimp.NET
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

[backend] Use a stack instead of queue when backfilling

Browse source 
This makes each reply chain load completely before loading the next
chain, instead of the current behavior that loads all replies of one
depth before loading the next "layer".

This won't make much of a difference *now*, but should result in more
intuitive behavior when live updating of newly loaded replies gets
implemented.
This commit is contained in:
Kopper 2024-11-06 13:37:19 +03:00 • committed by Iceshrimp development
parent 40b35a7bd2
commit 94328a3eef
1 changed files with 14 additions and 14 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
Iceshrimp.Backend/Core/Queues/BackfillQueue.cs
View file

@ -62,8 +62,8 @@ public class BackfillQueue(int parallelism)
.Select(n => new BackfillData(n.Id, n.RepliesCollection!))
.ToArrayAsync(token);
var toBackfill = new Queue<BackfillData>(toBackfillArray);
while (toBackfill.TryDequeue(out var currentItem))
var toBackfill = new Stack<BackfillData>(toBackfillArray);
while (toBackfill.TryPop(out var currentItem))
{
var current = currentItem;
if (!history.Add(current.RepliesCollection))
@ -108,7 +108,7 @@ public class BackfillQueue(int parallelism)
(note.RepliesFetchedAt == null ||
note.RepliesFetchedAt <= DateTime.UtcNow - cfg.RefreshAfterTimeSpan))
{
toBackfill.Enqueue(new BackfillData(note.Id, note.RepliesCollection!));
toBackfill.Push(new BackfillData(note.Id, note.RepliesCollection!));
}
}
catch (Exception e)