h1or4dme8/osu-lazer
1
0
Fork 0
mirror of https://github.com/ppy/osu.git synced 2025-02-13 10:33:07 +08:00
Code Issues Releases Wiki Activity

Ensure thread safety on shared contexts

Browse Source 
Let's call this one temporary.
This commit is contained in:
Dean Herbert 2017-10-23 17:56:04 +09:00
parent e7931ef4c7
commit 1a3debc91d
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
osu.Game/Database/DatabaseBackedStore.cs
View File

@ -2,6 +2,7 @@
// Licensed under the MIT Licence - https://raw.githubusercontent.com/ppy/osu/master/LICENCE
using System;
using System.Threading;
using osu.Framework.Logging;
using osu.Framework.Platform;
@ -16,7 +17,7 @@ namespace osu.Game.Database
/// </summary>
protected readonly Func<OsuDbContext> CreateContext;
private readonly Lazy<OsuDbContext> queryContext;
private readonly ThreadLocal<OsuDbContext> queryContext;
/// <summary>
/// Retrieve a shared context for performing lookups (or write operations on the update thread, for now).
@ -26,7 +27,9 @@ namespace osu.Game.Database
protected DatabaseBackedStore(Func<OsuDbContext> createContext, Storage storage = null)
{
CreateContext = createContext;
queryContext = new Lazy<OsuDbContext>(CreateContext);
// todo: while this seems to work quite well, we need to consider that contexts could enter a state where they are never cleaned up.
queryContext = new ThreadLocal<OsuDbContext>(CreateContext);
Storage = storage;