Skip to content

Commit c23f1f6

Browse files
author
Sophia Tevosyan
committed
added comments to the ExtendedSessionState class and the memory cache import to props
1 parent 38a6011 commit c23f1f6

File tree

2 files changed

+98
-82
lines changed

2 files changed

+98
-82
lines changed

‎Directory.Packages.props‎

Lines changed: 75 additions & 74 deletions
Original file line numberDiff line numberDiff line change
@@ -1,82 +1,83 @@
1-
<Project>
2-
<PropertyGroup>
1+
<Project>
2+
<PropertyGroup>
33
<!--
44
More information about central package management:
55
https://learn.microsoft.com/nuget/consume-packages/central-package-management
6-
-->
7-
<ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
8-
</PropertyGroup>
9-
10-
<!-- Microsoft.Extensions.* Packages -->
11-
<ItemGroup>
12-
<PackageVersion Include="Microsoft.Extensions.Configuration.Abstractions" Version="8.0.0" />
13-
<PackageVersion Include="Microsoft.Extensions.DependencyInjection" Version="8.0.1" />
14-
<PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="8.0.2" />
15-
<PackageVersion Include="Microsoft.Extensions.Hosting" Version="6.0.1" Condition="'$(TargetFramework)' == 'netstandard2.0' or '$(TargetFramework)' == 'net48'" />
6+
-->
7+
<ManagePackageVersionsCentrally>true</ManagePackageVersionsCentrally>
8+
</PropertyGroup>
9+
10+
<!-- Microsoft.Extensions.* Packages -->
11+
<ItemGroup>
12+
<PackageVersion Include="Microsoft.Extensions.Caching.Memory" Version="9.0.7" />
13+
<PackageVersion Include="Microsoft.Extensions.Configuration.Abstractions" Version="8.0.0" />
14+
<PackageVersion Include="Microsoft.Extensions.DependencyInjection" Version="8.0.1" />
15+
<PackageVersion Include="Microsoft.Extensions.DependencyInjection.Abstractions" Version="8.0.2" />
16+
<PackageVersion Include="Microsoft.Extensions.Hosting" Version="6.0.1" Condition="'$(TargetFramework)' == 'netstandard2.0' or '$(TargetFramework)' == 'net48'" />
1617
<PackageVersion Include="Microsoft.Extensions.Hosting" Version="8.0.1" Condition="'$(TargetFramework)' == 'net8.0'"/>
17-
<PackageVersion Include="Microsoft.Extensions.Hosting.Abstractions" Version="8.0.1" />
18-
<PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.3" />
19-
<PackageVersion Include="Microsoft.Extensions.Options" Version="8.0.2" />
20-
<PackageVersion Include="Microsoft.Extensions.Options.DataAnnotations" Version="8.0.0" />
21-
</ItemGroup>
22-
23-
<!-- Azure.* Packages -->
24-
<ItemGroup>
25-
<PackageVersion Include="Azure.Identity" Version="1.13.1" />
26-
<PackageVersion Include="Microsoft.Azure.Functions.Worker" Version="1.21.0" />
27-
</ItemGroup>
28-
29-
<!-- DurableTask Packages -->
30-
<ItemGroup>
31-
<PackageVersion Include="Microsoft.Azure.DurableTask.Core" Version="3.3.0" />
32-
<PackageVersion Include="Microsoft.Azure.Functions.Worker.Extensions.DurableTask" Version="1.2.2" />
33-
</ItemGroup>
34-
35-
<!-- Grpc / Protobuf Packages -->
36-
<ItemGroup>
37-
<PackageVersion Include="Google.Protobuf" Version="3.21.12" />
38-
<PackageVersion Include="Grpc.Core" Version="2.46.5" />
39-
<PackageVersion Include="Grpc.Net.Client" Version="2.67.0" />
40-
<PackageVersion Include="Grpc.Tools" Version="2.51.0" />
41-
<PackageVersion Include="Grpc.AspNetCore.Server" Version="2.52.0" />
42-
</ItemGroup>
43-
44-
<!-- Microsoft.CodeAnalysis.* Packages -->
45-
<ItemGroup>
46-
<PackageVersion Include="Microsoft.CodeAnalysis" Version="4.9.2" />
47-
<PackageVersion Include="Microsoft.CodeAnalysis.Analyzers" Version="3.3.4" />
48-
<PackageVersion Include="Microsoft.CodeAnalysis.CSharp" Version="4.9.2" />
49-
<PackageVersion Include="Microsoft.CodeAnalysis.CSharp.Workspaces" Version="4.9.2" />
50-
<PackageVersion Include="Microsoft.CodeAnalysis.Common" Version="4.9.2" />
51-
<PackageVersion Include="Microsoft.CodeAnalysis.CSharp.Analyzer.Testing" Version="1.1.2" />
52-
<PackageVersion Include="Microsoft.CodeAnalysis.CSharp.CodeFix.Testing" Version="1.1.2" />
53-
<PackageVersion Include="Microsoft.CodeAnalysis.CSharp.SourceGenerators.Testing" Version="1.1.2" />
54-
<PackageVersion Include="Microsoft.CodeAnalysis.ResxSourceGenerator" Version="3.11.0-beta1.24165.2" />
55-
</ItemGroup>
56-
57-
<!-- Testing Frameworks & Analysis Packages -->
58-
<ItemGroup>
59-
<PackageVersion Include="BenchmarkDotNet" Version="0.13.2" />
60-
<PackageVersion Include="FluentAssertions" Version="6.7.0" />
18+
<PackageVersion Include="Microsoft.Extensions.Hosting.Abstractions" Version="8.0.1" />
19+
<PackageVersion Include="Microsoft.Extensions.Logging.Abstractions" Version="8.0.3" />
20+
<PackageVersion Include="Microsoft.Extensions.Options" Version="8.0.2" />
21+
<PackageVersion Include="Microsoft.Extensions.Options.DataAnnotations" Version="8.0.0" />
22+
</ItemGroup>
23+
24+
<!-- Azure.* Packages -->
25+
<ItemGroup>
26+
<PackageVersion Include="Azure.Identity" Version="1.13.1" />
27+
<PackageVersion Include="Microsoft.Azure.Functions.Worker" Version="1.21.0" />
28+
</ItemGroup>
29+
30+
<!-- DurableTask Packages -->
31+
<ItemGroup>
32+
<PackageVersion Include="Microsoft.Azure.DurableTask.Core" Version="3.3.0" />
33+
<PackageVersion Include="Microsoft.Azure.Functions.Worker.Extensions.DurableTask" Version="1.2.2" />
34+
</ItemGroup>
35+
36+
<!-- Grpc / Protobuf Packages -->
37+
<ItemGroup>
38+
<PackageVersion Include="Google.Protobuf" Version="3.21.12" />
39+
<PackageVersion Include="Grpc.Core" Version="2.46.5" />
40+
<PackageVersion Include="Grpc.Net.Client" Version="2.67.0" />
41+
<PackageVersion Include="Grpc.Tools" Version="2.51.0" />
42+
<PackageVersion Include="Grpc.AspNetCore.Server" Version="2.52.0" />
43+
</ItemGroup>
44+
45+
<!-- Microsoft.CodeAnalysis.* Packages -->
46+
<ItemGroup>
47+
<PackageVersion Include="Microsoft.CodeAnalysis" Version="4.9.2" />
48+
<PackageVersion Include="Microsoft.CodeAnalysis.Analyzers" Version="3.3.4" />
49+
<PackageVersion Include="Microsoft.CodeAnalysis.CSharp" Version="4.9.2" />
50+
<PackageVersion Include="Microsoft.CodeAnalysis.CSharp.Workspaces" Version="4.9.2" />
51+
<PackageVersion Include="Microsoft.CodeAnalysis.Common" Version="4.9.2" />
52+
<PackageVersion Include="Microsoft.CodeAnalysis.CSharp.Analyzer.Testing" Version="1.1.2" />
53+
<PackageVersion Include="Microsoft.CodeAnalysis.CSharp.CodeFix.Testing" Version="1.1.2" />
54+
<PackageVersion Include="Microsoft.CodeAnalysis.CSharp.SourceGenerators.Testing" Version="1.1.2" />
55+
<PackageVersion Include="Microsoft.CodeAnalysis.ResxSourceGenerator" Version="3.11.0-beta1.24165.2" />
56+
</ItemGroup>
57+
58+
<!-- Testing Frameworks & Analysis Packages -->
59+
<ItemGroup>
60+
<PackageVersion Include="BenchmarkDotNet" Version="0.13.2" />
61+
<PackageVersion Include="FluentAssertions" Version="6.7.0" />
6162
<PackageVersion Include="FluentAssertions.Analyzers" Version="0.17.2"/>
62-
<PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
63-
<PackageVersion Include="Moq" Version="4.20.70" />
64-
<PackageVersion Include="StyleCop.Analyzers" Version="1.2.0-beta.556" />
65-
<PackageVersion Include="coverlet.collector" Version="3.1.2" />
66-
<PackageVersion Include="xunit" Version="2.5.3" />
67-
<PackageVersion Include="xunit.abstractions" Version="2.0.3" />
63+
<PackageVersion Include="Microsoft.NET.Test.Sdk" Version="17.8.0" />
64+
<PackageVersion Include="Moq" Version="4.20.70" />
65+
<PackageVersion Include="StyleCop.Analyzers" Version="1.2.0-beta.556" />
66+
<PackageVersion Include="coverlet.collector" Version="3.1.2" />
67+
<PackageVersion Include="xunit" Version="2.5.3" />
68+
<PackageVersion Include="xunit.abstractions" Version="2.0.3" />
6869
<PackageVersion Include="xunit.runner.visualstudio" Version="2.5.3"/>
69-
<PackageVersion Include="Xunit.Combinatorial" Version="1.6.24" />
70-
</ItemGroup>
71-
72-
<!-- Base-class library dependencies -->
73-
<ItemGroup>
74-
<PackageVersion Include="DotNext" Version="4.13.1" />
75-
<PackageVersion Include="Microsoft.SourceLink.GitHub" Version="1.1.1" />
76-
<PackageVersion Include="Microsoft.Bcl.AsyncInterfaces" Version="8.0.0" />
77-
<PackageVersion Include="System.Linq.Async" Version="6.0.3" />
78-
<PackageVersion Include="System.Text.Json" Version="8.0.6" />
79-
<PackageVersion Include="System.Collections.Immutable" Version="8.0.0" />
80-
</ItemGroup>
70+
<PackageVersion Include="Xunit.Combinatorial" Version="1.6.24" />
71+
</ItemGroup>
72+
73+
<!-- Base-class library dependencies -->
74+
<ItemGroup>
75+
<PackageVersion Include="DotNext" Version="4.13.1" />
76+
<PackageVersion Include="Microsoft.SourceLink.GitHub" Version="1.1.1" />
77+
<PackageVersion Include="Microsoft.Bcl.AsyncInterfaces" Version="8.0.0" />
78+
<PackageVersion Include="System.Linq.Async" Version="6.0.3" />
79+
<PackageVersion Include="System.Text.Json" Version="8.0.6" />
80+
<PackageVersion Include="System.Collections.Immutable" Version="8.0.0" />
81+
</ItemGroup>
8182

8283
</Project>

‎src/Worker/Grpc/ExtendedSessionState.cs‎

Lines changed: 23 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -6,20 +6,35 @@
66
namespace Microsoft.DurableTask.Worker.Grpc;
77

88
/// <summary>
9-
///
9+
/// Represents the state of an extended session for an orchestration.
1010
/// </summary>
1111
class ExtendedSessionState
1212
{
13+
/// <summary>
14+
/// Initializes a new instance of the <see cref="ExtendedSessionState"/> class.
15+
/// </summary>
16+
/// <param name="state">The orchestration's runtime state.</param>
17+
/// <param name="taskOrchestration">The TaskOrchestration implementation of the orchestration.</param>
18+
/// <param name="orchestrationExecutor">The TaskOrchestrationExecutor for the orchestration.</param>
19+
internal ExtendedSessionState(OrchestrationRuntimeState state, TaskOrchestration taskOrchestration, TaskOrchestrationExecutor orchestrationExecutor)
20+
{
21+
this.RuntimeState = state;
22+
this.TaskOrchestration = taskOrchestration;
23+
this.OrchestrationExecutor = orchestrationExecutor;
24+
}
25+
26+
/// <summary>
27+
/// Gets or sets the saved runtime state of the orchestration.
28+
/// </summary>
1329
internal OrchestrationRuntimeState RuntimeState { get; set; }
1430

31+
/// <summary>
32+
/// Gets or sets the saved TaskOrchestration implementation of the orchestration.
33+
/// </summary>
1534
internal TaskOrchestration TaskOrchestration { get; set; }
1635

36+
/// <summary>
37+
/// Gets or sets the saved TaskOrchestrationExecutor.
38+
/// </summary>
1739
internal TaskOrchestrationExecutor OrchestrationExecutor { get; set; }
18-
19-
public ExtendedSessionState(OrchestrationRuntimeState state, TaskOrchestration taskOrchestration, TaskOrchestrationExecutor orchestrationExecutor)
20-
{
21-
RuntimeState = state;
22-
TaskOrchestration = taskOrchestration;
23-
OrchestrationExecutor = orchestrationExecutor;
24-
}
2540
}

0 commit comments

Comments
 (0)