mirror of
https://github.com/jellyfin/jellyfin.git
synced 2026-04-13 05:42:16 +01:00
Some checks failed
Stale PR Check / Check PRs with merge conflicts (push) Has been cancelled
CodeQL / Analyze (csharp) (push) Has been cancelled
OpenAPI / OpenAPI - HEAD (push) Has been cancelled
OpenAPI / OpenAPI - BASE (push) Has been cancelled
Tests / run-tests (macos-latest) (push) Has been cancelled
Tests / run-tests (ubuntu-latest) (push) Has been cancelled
Tests / run-tests (windows-latest) (push) Has been cancelled
Project Automation / Project board (push) Has been cancelled
Merge Conflict Labeler / Labeling (push) Has been cancelled
OpenAPI / OpenAPI - Difference (push) Has been cancelled
OpenAPI / OpenAPI - Publish Unstable Spec (push) Has been cancelled
OpenAPI / OpenAPI - Publish Stable Spec (push) Has been cancelled
Stale Issue Labeler / Check for stale issues (push) Has been cancelled
fix: in optimistic locking, key off table is locked
Original-merge: b5f0199a25
Merged-by: crobibero <cody@robibe.ro>
Backported-by: Joshua M. Boniface <joshua@boniface.me>
162 lines
7.6 KiB
C#
162 lines
7.6 KiB
C#
using System;
|
|
using System.Data.Common;
|
|
using System.Linq;
|
|
using System.Security.Cryptography;
|
|
using System.Threading;
|
|
using System.Threading.Tasks;
|
|
using Microsoft.EntityFrameworkCore;
|
|
using Microsoft.EntityFrameworkCore.Diagnostics;
|
|
using Microsoft.Extensions.Logging;
|
|
using Polly;
|
|
|
|
namespace Jellyfin.Database.Implementations.Locking;
|
|
|
|
/// <summary>
|
|
/// Defines a locking mechanism that will retry any write operation for a few times.
|
|
/// </summary>
|
|
public class OptimisticLockBehavior : IEntityFrameworkCoreLockingBehavior
|
|
{
|
|
private readonly Policy _writePolicy;
|
|
private readonly AsyncPolicy _writeAsyncPolicy;
|
|
private readonly ILogger<OptimisticLockBehavior> _logger;
|
|
|
|
/// <summary>
|
|
/// Initializes a new instance of the <see cref="OptimisticLockBehavior"/> class.
|
|
/// </summary>
|
|
/// <param name="logger">The application logger.</param>
|
|
public OptimisticLockBehavior(ILogger<OptimisticLockBehavior> logger)
|
|
{
|
|
TimeSpan[] sleepDurations = [
|
|
TimeSpan.FromMilliseconds(50),
|
|
TimeSpan.FromMilliseconds(50),
|
|
TimeSpan.FromMilliseconds(50),
|
|
TimeSpan.FromMilliseconds(50),
|
|
TimeSpan.FromMilliseconds(250),
|
|
TimeSpan.FromMilliseconds(250),
|
|
TimeSpan.FromMilliseconds(250),
|
|
TimeSpan.FromMilliseconds(150),
|
|
TimeSpan.FromMilliseconds(150),
|
|
TimeSpan.FromMilliseconds(150),
|
|
TimeSpan.FromMilliseconds(500),
|
|
TimeSpan.FromMilliseconds(150),
|
|
TimeSpan.FromMilliseconds(500),
|
|
TimeSpan.FromMilliseconds(150),
|
|
TimeSpan.FromSeconds(3)
|
|
];
|
|
|
|
Func<int, Context, TimeSpan> backoffProvider = (index, context) =>
|
|
{
|
|
var backoff = sleepDurations[index];
|
|
return backoff + TimeSpan.FromMilliseconds(RandomNumberGenerator.GetInt32(0, (int)(backoff.TotalMilliseconds * .5)));
|
|
};
|
|
|
|
_logger = logger;
|
|
_writePolicy = Policy
|
|
.HandleInner<Exception>(e =>
|
|
e.Message.Contains("database is locked", StringComparison.InvariantCultureIgnoreCase) ||
|
|
e.Message.Contains("database table is locked", StringComparison.InvariantCultureIgnoreCase))
|
|
.WaitAndRetry(sleepDurations.Length, backoffProvider, RetryHandle);
|
|
_writeAsyncPolicy = Policy
|
|
.HandleInner<Exception>(e =>
|
|
e.Message.Contains("database is locked", StringComparison.InvariantCultureIgnoreCase) ||
|
|
e.Message.Contains("database table is locked", StringComparison.InvariantCultureIgnoreCase))
|
|
.WaitAndRetryAsync(sleepDurations.Length, backoffProvider, RetryHandle);
|
|
|
|
void RetryHandle(Exception exception, TimeSpan timespan, int retryNo, Context context)
|
|
{
|
|
if (retryNo < sleepDurations.Length)
|
|
{
|
|
_logger.LogWarning("Operation failed retry {RetryNo}", retryNo);
|
|
}
|
|
else
|
|
{
|
|
_logger.LogError(exception, "Operation failed retry {RetryNo}", retryNo);
|
|
}
|
|
}
|
|
}
|
|
|
|
/// <inheritdoc/>
|
|
public void Initialise(DbContextOptionsBuilder optionsBuilder)
|
|
{
|
|
_logger.LogInformation("The database locking mode has been set to: Optimistic.");
|
|
optionsBuilder.AddInterceptors(new RetryInterceptor(_writeAsyncPolicy, _writePolicy));
|
|
optionsBuilder.AddInterceptors(new TransactionLockingInterceptor(_writeAsyncPolicy, _writePolicy));
|
|
}
|
|
|
|
/// <inheritdoc/>
|
|
public void OnSaveChanges(JellyfinDbContext context, Action saveChanges)
|
|
{
|
|
_writePolicy.ExecuteAndCapture(saveChanges);
|
|
}
|
|
|
|
/// <inheritdoc/>
|
|
public async Task OnSaveChangesAsync(JellyfinDbContext context, Func<Task> saveChanges)
|
|
{
|
|
await _writeAsyncPolicy.ExecuteAndCaptureAsync(saveChanges).ConfigureAwait(false);
|
|
}
|
|
|
|
private sealed class TransactionLockingInterceptor : DbTransactionInterceptor
|
|
{
|
|
private readonly AsyncPolicy _asyncRetryPolicy;
|
|
private readonly Policy _retryPolicy;
|
|
|
|
public TransactionLockingInterceptor(AsyncPolicy asyncRetryPolicy, Policy retryPolicy)
|
|
{
|
|
_asyncRetryPolicy = asyncRetryPolicy;
|
|
_retryPolicy = retryPolicy;
|
|
}
|
|
|
|
public override InterceptionResult<DbTransaction> TransactionStarting(DbConnection connection, TransactionStartingEventData eventData, InterceptionResult<DbTransaction> result)
|
|
{
|
|
return InterceptionResult<DbTransaction>.SuppressWithResult(_retryPolicy.Execute(() => connection.BeginTransaction(eventData.IsolationLevel)));
|
|
}
|
|
|
|
public override async ValueTask<InterceptionResult<DbTransaction>> TransactionStartingAsync(DbConnection connection, TransactionStartingEventData eventData, InterceptionResult<DbTransaction> result, CancellationToken cancellationToken = default)
|
|
{
|
|
return InterceptionResult<DbTransaction>.SuppressWithResult(await _asyncRetryPolicy.ExecuteAsync(async () => await connection.BeginTransactionAsync(eventData.IsolationLevel, cancellationToken).ConfigureAwait(false)).ConfigureAwait(false));
|
|
}
|
|
}
|
|
|
|
private sealed class RetryInterceptor : DbCommandInterceptor
|
|
{
|
|
private readonly AsyncPolicy _asyncRetryPolicy;
|
|
private readonly Policy _retryPolicy;
|
|
|
|
public RetryInterceptor(AsyncPolicy asyncRetryPolicy, Policy retryPolicy)
|
|
{
|
|
_asyncRetryPolicy = asyncRetryPolicy;
|
|
_retryPolicy = retryPolicy;
|
|
}
|
|
|
|
public override InterceptionResult<int> NonQueryExecuting(DbCommand command, CommandEventData eventData, InterceptionResult<int> result)
|
|
{
|
|
return InterceptionResult<int>.SuppressWithResult(_retryPolicy.Execute(command.ExecuteNonQuery));
|
|
}
|
|
|
|
public override async ValueTask<InterceptionResult<int>> NonQueryExecutingAsync(DbCommand command, CommandEventData eventData, InterceptionResult<int> result, CancellationToken cancellationToken = default)
|
|
{
|
|
return InterceptionResult<int>.SuppressWithResult(await _asyncRetryPolicy.ExecuteAsync(async () => await command.ExecuteNonQueryAsync(cancellationToken).ConfigureAwait(false)).ConfigureAwait(false));
|
|
}
|
|
|
|
public override InterceptionResult<object> ScalarExecuting(DbCommand command, CommandEventData eventData, InterceptionResult<object> result)
|
|
{
|
|
return InterceptionResult<object>.SuppressWithResult(_retryPolicy.Execute(() => command.ExecuteScalar()!));
|
|
}
|
|
|
|
public override async ValueTask<InterceptionResult<object>> ScalarExecutingAsync(DbCommand command, CommandEventData eventData, InterceptionResult<object> result, CancellationToken cancellationToken = default)
|
|
{
|
|
return InterceptionResult<object>.SuppressWithResult((await _asyncRetryPolicy.ExecuteAsync(async () => await command.ExecuteScalarAsync(cancellationToken).ConfigureAwait(false)!).ConfigureAwait(false))!);
|
|
}
|
|
|
|
public override InterceptionResult<DbDataReader> ReaderExecuting(DbCommand command, CommandEventData eventData, InterceptionResult<DbDataReader> result)
|
|
{
|
|
return InterceptionResult<DbDataReader>.SuppressWithResult(_retryPolicy.Execute(command.ExecuteReader));
|
|
}
|
|
|
|
public override async ValueTask<InterceptionResult<DbDataReader>> ReaderExecutingAsync(DbCommand command, CommandEventData eventData, InterceptionResult<DbDataReader> result, CancellationToken cancellationToken = default)
|
|
{
|
|
return InterceptionResult<DbDataReader>.SuppressWithResult(await _asyncRetryPolicy.ExecuteAsync(async () => await command.ExecuteReaderAsync(cancellationToken).ConfigureAwait(false)).ConfigureAwait(false));
|
|
}
|
|
}
|
|
}
|