Skip to content

Commit

Permalink
Rename fields, add comments
Browse files Browse the repository at this point in the history
  • Loading branch information
antlai-temporal committed Dec 5, 2024
1 parent 33b91c8 commit f81297c
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 7 deletions.
8 changes: 4 additions & 4 deletions internal/deployment_client.go
Original file line number Diff line number Diff line change
Expand Up @@ -99,8 +99,8 @@ type (
// CreateTime - When this deployment was created.
CreateTime time.Time

// Current - Whether this deployment is the current one for its deployment series.
Current bool
// IsCurrent - Whether this deployment is the current one for its deployment series.
IsCurrent bool

// TaskQueuesInfo - List of task queues polled by workers in this deployment.
TaskQueuesInfo []DeploymentTaskQueueInfo
Expand All @@ -119,7 +119,7 @@ type (
CreateTime time.Time

// Whether this deployment is the current one for its deployment series.
Current bool
IsCurrent bool
}

// DeploymentListIterator is an iterator for deployments.
Expand Down Expand Up @@ -147,7 +147,7 @@ type (
// DeploymentReachabilityInfo extends DeploymentInfo with reachability information.
// NOTE: Experimental
DeploymentReachabilityInfo struct {
DeploymentInfo
DeploymentInfo DeploymentInfo

// Reachability - Kind of tasks that may reach a worker
// associated with a deployment.
Expand Down
4 changes: 2 additions & 2 deletions internal/internal_deployment_client.go
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@ func deploymentListEntryFromProto(deployment *deployment.DeploymentListInfo) *De
return &DeploymentListEntry{
Deployment: deploymentFromProto(deployment.GetDeployment()),
CreateTime: deployment.GetCreateTime().AsTime(),
Current: deployment.GetIsCurrent(),
IsCurrent: deployment.GetIsCurrent(),
}
}

Expand All @@ -115,7 +115,7 @@ func deploymentInfoFromProto(deploymentInfo *deployment.DeploymentInfo) Deployme
return DeploymentInfo{
Deployment: deploymentFromProto(deploymentInfo.GetDeployment()),
CreateTime: deploymentInfo.GetCreateTime().AsTime(),
Current: deploymentInfo.GetIsCurrent(),
IsCurrent: deploymentInfo.GetIsCurrent(),
TaskQueuesInfo: deploymentTaskQueuesInfoFromProto(deploymentInfo.GetTaskQueueInfos()),
Metadata: deploymentInfo.GetMetadata(),
}
Expand Down
4 changes: 3 additions & 1 deletion internal/workflow.go
Original file line number Diff line number Diff line change
Expand Up @@ -413,7 +413,9 @@ type (
// inside a workflow as a child workflow.
Name string
DisableAlreadyRegisteredCheck bool
// Optional: Provides a Versioning Behavior to workflows of this type.
// Optional: Provides a Versioning Behavior to workflows of this type. It is required
// when WorkerOptions does not specify DeploymentOptions.DefaultVersioningBehavior,
// DeploymentOptions.DeploymentSeriesName is set, and UseBuildIDForVersioning is true.
// NOTE: Experimental
VersioningBehavior VersioningBehavior
}
Expand Down

0 comments on commit f81297c

Please sign in to comment.