h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2026-05-25 03:49:55 +08:00
Code Issues Releases Wiki Activity

Remove RealmLive context re-fetch optimisation for now

Browse Source
This commit is contained in:
Dean Herbert
2021-11-26 14:43:35 +09:00
Unverified
parent bb6f8bd59c
commit e40e5096ea
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
osu.Game/Database/RealmLive.cs
+4 -1
View File
@@ -114,7 +114,10 @@ namespace osu.Game.Database
}
}
private bool originalDataValid => !IsManaged || (isCorrectThread && data.IsValid);
// TODO: Revisit adding these conditionals back as an optimisation: || (isCorrectThread && data.IsValid);
// They have temporarily been removed due to an oversight involving .AsQueryable, see https://github.com/realm/realm-dotnet/discussions/2734.
// This means we are fetching a new context every `PerformRead` or `PerformWrite`, even when on the correct thread.
private bool originalDataValid => !IsManaged;
// this matches realm's internal thread validation (see https://github.com/realm/realm-dotnet/blob/903b4d0b304f887e37e2d905384fb572a6496e70/Realm/Realm/Native/SynchronizationContextScheduler.cs#L72)
private bool isCorrectThread