fix missing await for task in ApplyToStore (#112)

* fix missing await for task in ApplyToStore

* increase timeout for test to fix CI test fail
This commit is contained in:
Sebastian Burckhardt 2022-01-21 14:54:33 -08:00 коммит произвёл GitHub
Родитель ab89349e3e
Коммит 94129ab595
Не найден ключ, соответствующий данной подписи
Идентификатор ключа GPG: 4AEE18F83AFDEB23
2 изменённых файлов: 2 добавлений и 3 удалений

Просмотреть файл

@ -23,8 +23,7 @@ namespace DurableTask.Netherite.Faster
public override ValueTask ApplyToStore(TrackedObjectKey key, EffectTracker tracker)
{
this.store.ProcessEffectOnTrackedObject(key, tracker);
return default;
return this.store.ProcessEffectOnTrackedObject(key, tracker);
}
public override (long, long) GetPositions()

Просмотреть файл

@ -193,7 +193,7 @@ namespace DurableTask.Netherite.Tests
public async Task LargeFanOutOrchestration()
{
var client = await this.host.StartOrchestrationAsync(typeof(Orchestrations.FanOutFanIn), 1000);
var status = await client.WaitForCompletionAsync(TimeSpan.FromMinutes(5));
var status = await client.WaitForCompletionAsync(TimeSpan.FromMinutes(8));
Assert.Equal(OrchestrationStatus.Completed, status?.OrchestrationStatus);
}