udpate by comment

Author: nytian

src/Abstractions/TaskOrchestrationContext.cs

@@ -68,7 +68,7 @@ public abstract class TaskOrchestrationContext
     /// <summary>
     /// Gets the configuration settings for the orchestration context.
     /// </summary>
-    public abstract IDictionary<string, object?> Properties { get; }
+    public virtual IDictionary<string, object?> Properties { get; } = new Dictionary<string, object?>();
 
     /// <summary>
     /// Gets the entity feature, for interacting with entities.

src/Worker/Core/Shims/TaskOrchestrationContextWrapper.cs

@@ -23,7 +23,6 @@ sealed partial class TaskOrchestrationContextWrapper : TaskOrchestrationContext
     readonly OrchestrationInvocationContext invocationContext;
     readonly ILogger logger;
     readonly object? deserializedInput;
-    readonly Dictionary<string, object?> properties = new Dictionary<string, object?>();
 
     int newGuidCounter;
     object? customStatus;
@@ -39,11 +38,8 @@ public TaskOrchestrationContextWrapper(
         OrchestrationContext innerContext,
         OrchestrationInvocationContext invocationContext,
         object? deserializedInput)
+        : this(innerContext, invocationContext, deserializedInput, new Dictionary<string, object?>())
     {
-        this.innerContext = Check.NotNull(innerContext);
-        this.invocationContext = Check.NotNull(invocationContext);
-        this.logger = this.CreateReplaySafeLogger("Microsoft.DurableTask");
-        this.deserializedInput = deserializedInput;
     }
 
     /// <summary>
@@ -57,11 +53,16 @@ public TaskOrchestrationContextWrapper(
         OrchestrationContext innerContext,
         OrchestrationInvocationContext invocationContext,
         object? deserializedInput,
-        Dictionary<string, object?> properties)
+        IEnumerable<KeyValuePair<string, object?>> properties)
     {
         this.innerContext = Check.NotNull(innerContext);
         this.invocationContext = Check.NotNull(invocationContext);
-        this.properties = Check.NotNull(properties);
+        if (properties is null)
+        {
+            throw new ArgumentNullException(nameof(properties));
+        }
+
+        this.Properties = properties.ToDictionary(pair => pair.Key, pair => pair.Value);
         this.logger = this.CreateReplaySafeLogger("Microsoft.DurableTask");
         this.deserializedInput = deserializedInput;
     }
@@ -84,7 +85,7 @@ public TaskOrchestrationContextWrapper(
     /// <summary>
     /// Gets the configuration settings for the orchestration.
     /// </summary>
-    public override IDictionary<string, object?> Properties => this.properties;
+    public override IDictionary<string, object?> Properties { get; } = new Dictionary<string, object?>();
 
     /// <inheritdoc/>
     public override TaskOrchestrationEntityFeature Entities

src/Worker/Core/Shims/TaskOrchestrationShim.cs

@@ -31,11 +31,8 @@ partial class TaskOrchestrationShim : TaskOrchestration
     public TaskOrchestrationShim(
         OrchestrationInvocationContext invocationContext,
         ITaskOrchestrator implementation)
+        : this(invocationContext, implementation, new Dictionary<string, object?>())
     {
-        this.invocationContext = Check.NotNull(invocationContext);
-        this.implementation = Check.NotNull(implementation);
-
-        this.logger = Logs.CreateWorkerLogger(this.invocationContext.LoggerFactory, "Orchestrations");
     }
 
     /// <summary>