Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
// -----------------------------------------------------------------------
// <copyright file="SystemDataSqliteSnapshotSpec.cs" company="Akka.NET Project">
// Copyright (C) 2013-2023 .NET Foundation <https://github.com/akkadotnet/akka.net>
// </copyright>
// -----------------------------------------------------------------------

using Akka.Persistence.Sql.Tests.Common.Containers;
using Akka.Persistence.TCK.Snapshot;
using Xunit;
using Xunit.Abstractions;

namespace Akka.Persistence.Sql.Tests.Sqlite
{
[Collection(nameof(SqlitePersistenceSpec))]
public class SystemDataSqliteSnapshotSpec: SnapshotStoreSpec
{
public SystemDataSqliteSnapshotSpec(ITestOutputHelper output, SqliteContainer fixture)
: base(SqliteSnapshotSpecConfig.Create(fixture), nameof(SystemDataSqliteSnapshotSpec), output)
{
Initialize();
}
}
}
26 changes: 18 additions & 8 deletions src/Akka.Persistence.Sql/Snapshot/ByteArraySnapshotDao.cs
Original file line number Diff line number Diff line change
Expand Up @@ -357,6 +357,7 @@ public async Task<Option<SelectedSnapshot>> SnapshotForMaxSequenceNrAndMaxTimest
public async Task DeleteAsync(
string persistenceId,
long sequenceNr,
DateTime timestamp,
CancellationToken cancellationToken = default)
{
var cts = CancellationTokenSource.CreateLinkedTokenSource(cancellationToken, _shutdownCts.Token);
Expand All @@ -367,22 +368,31 @@ await _connectionFactory.ExecuteWithTransactionAsync(
{
if (connection.UseDateTime)
{
await connection
.GetTable<DateTimeSnapshotRow>()
.Where(
r =>
r.PersistenceId == persistenceId &&
r.SequenceNumber == sequenceNr)
var query = connection.GetTable<DateTimeSnapshotRow>()
.Where(
r =>
r.PersistenceId == persistenceId &&
r.SequenceNumber == sequenceNr);

if (timestamp > DateTime.MinValue)
query = query.Where(r => r.Created <= timestamp);

await query
.DeleteAsync(token);
}
else
{
await connection
var query = connection
.GetTable<LongSnapshotRow>()
.Where(
r =>
r.PersistenceId == persistenceId &&
r.SequenceNumber == sequenceNr)
r.SequenceNumber == sequenceNr);

if (timestamp > DateTime.MinValue)
query = query.Where(r => r.Created <= timestamp.Ticks);

await query
.DeleteAsync(token);
}
});
Expand Down
1 change: 1 addition & 0 deletions src/Akka.Persistence.Sql/Snapshot/ISnapshotDao.cs
Original file line number Diff line number Diff line change
Expand Up @@ -56,6 +56,7 @@ Task<Option<SelectedSnapshot>> SnapshotForMaxSequenceNrAndMaxTimestampAsync(
Task DeleteAsync(
string persistenceId,
long sequenceNr,
DateTime timestamp,
CancellationToken cancellationToken = default);

Task SaveAsync(
Expand Down
2 changes: 1 addition & 1 deletion src/Akka.Persistence.Sql/Snapshot/SqlSnapshotStore.cs
Original file line number Diff line number Diff line change
Expand Up @@ -114,7 +114,7 @@ protected override async Task SaveAsync(SnapshotMetadata metadata, object snapsh
=> await _dao.SaveAsync(metadata, snapshot);

protected override async Task DeleteAsync(SnapshotMetadata metadata)
=> await _dao.DeleteAsync(metadata.PersistenceId, metadata.SequenceNr);
=> await _dao.DeleteAsync(metadata.PersistenceId, metadata.SequenceNr, metadata.Timestamp);

protected override async Task DeleteAsync(string persistenceId, SnapshotSelectionCriteria criteria)
{
Expand Down
2 changes: 1 addition & 1 deletion src/Directory.Packages.props
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<Project>
<PropertyGroup>
<ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
<AkkaVersion>1.5.27.1</AkkaVersion>
<AkkaVersion>1.5.28-beta1</AkkaVersion>
<AkkaHostingVersion>1.5.27</AkkaHostingVersion>
<FluentMigratorVersion>5.2.0</FluentMigratorVersion>
<SqliteVersion>1.0.118</SqliteVersion>
Expand Down