Add TimestampedSetRepository #2

Merged
on.nemtina merged 8 commits from TimestampedSetRepository into master 2024-11-26 12:59:05 +05:00
Collaborator

Доку допишу завтра

Доку допишу завтра
ng.frolov added 1 commit 2024-11-22 17:58:45 +05:00
ng.frolov added 1 commit 2024-11-25 09:41:12 +05:00
ng.frolov requested review from on.nemtina 2024-11-25 09:41:28 +05:00
ng.frolov added 1 commit 2024-11-25 09:41:51 +05:00
ng.frolov added 1 commit 2024-11-25 17:17:59 +05:00
ng.frolov added 1 commit 2024-11-26 10:07:50 +05:00
Поправлены тесты.
Удален не используемый интерфейс IPersistenceDbContext.
on.nemtina requested changes 2024-11-26 10:43:59 +05:00
@ -0,0 +20,4 @@
this.db = db;
}
public Task<int> InsertRange(Guid idDiscriminator, IEnumerable<TimestampedSetDto> sets, CancellationToken token)
Owner

Async и await здесь не нужен?

Async и await здесь не нужен?
@ -0,0 +60,4 @@
var data = await Materialize(query, token);
if (props is not null && props.Any())
Owner

Фильтр применился после материализации, а материализовывается только take элементов. А вдруг именно в этой порции ничего не найдено?

Фильтр применился после материализации, а материализовывается только take элементов. А вдруг именно в этой порции ничего не найдено?
@ -0,0 +66,4 @@
return data;
}
public Task<int> Count(Guid idDiscriminator, CancellationToken token)
Owner

Async и await здесь не нужен?

Async и await здесь не нужен?
on.nemtina requested changes 2024-11-26 11:02:36 +05:00
@ -0,0 +107,4 @@
return query.Where(entity => entity.Timestamp >= geTimestampUtc);
}
private static IEnumerable<TimestampedSetDto> ApplyPropsFilter(IEnumerable<TimestampedSetDto> query, IEnumerable<string> props)
Owner

Переименовать (убрать слово Filter)

Переименовать (убрать слово Filter)
ng.frolov added 1 commit 2024-11-26 11:24:32 +05:00
Fix TimestampedSetController doc and response type;
ITimestampedSetClient Add doc.
ng.frolov added 2 commits 2024-11-26 11:24:57 +05:00
on.nemtina merged commit fdf49b91ab into master 2024-11-26 12:59:05 +05:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: on.nemtina/persistence#2
No description provided.