Skip to content

Commit 1bc64c9

Browse files
committed
refined the comment for WorkflowExecutionOptions
1 parent 1f73c30 commit 1bc64c9

4 files changed

Lines changed: 25 additions & 17 deletions

File tree

openapi/openapiv2.json

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21009,7 +21009,7 @@
2100921009
},
2101021010
"timeSkippingConfig": {
2101121011
"$ref": "#/definitions/v1TimeSkippingConfig",
21012-
"description": "Time-skipping configuration for this workflow execution.\nIf not set, the time-skipping configuration is not updated by this request;\nthe existing configuration is preserved.\n\nWhen `fast_forward` is set, time will be fast-forwarded to a future point relative\nto the current workflow timestamp. Each call takes effect, even if\n`fast_forward` is set to the same duration, since the target time is recalculated\nfrom the current timestamp on every call."
21012+
"description": "The time-skipping configuration for this workflow execution.\nWhen `fast_forward` is set, time will be fast-forwarded to a future point relative\nto the current workflow timestamp. Each call takes effect, even if\n`fast_forward` is set to the same duration, since the target time is recalculated\nfrom the current timestamp on every call.\n\nThis field must be updated as a whole; updating individual sub-fields is not supported.\nWhen setting the update mask in `UpdateWorkflowExecutionOptionsRequest`, \n`BatchOperationUpdateWorkflowExecutionOptions`, etc., use a mask that covers the entire field."
2101321013
}
2101421014
}
2101521015
},
@@ -21046,7 +21046,11 @@
2104621046
},
2104721047
"timeSkippingConfig": {
2104821048
"$ref": "#/definitions/v1TimeSkippingConfig",
21049-
"description": "If set, the time-skipping configuration was changed. Contains the full updated configuration."
21049+
"description": "TimeSkippingConfig override upserted in this event. Represents the full config."
21050+
},
21051+
"timeSkippingConfigUpdated": {
21052+
"type": "boolean",
21053+
"description": "Indicates the time skipping config was updated by the recent call to update\nworkflow execution options."
2105021054
},
2105121055
"workflowUpdateOptions": {
2105221056
"type": "array",

openapi/openapiv3.yaml

Lines changed: 7 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -19534,15 +19534,7 @@ components:
1953419534
timeSkippingConfig:
1953519535
allOf:
1953619536
- $ref: '#/components/schemas/TimeSkippingConfig'
19537-
description: |-
19538-
Time-skipping configuration for this workflow execution.
19539-
If not set, the time-skipping configuration is not updated by this request;
19540-
the existing configuration is preserved.
19541-
19542-
When `fast_forward` is set, time will be fast-forwarded to a future point relative
19543-
to the current workflow timestamp. Each call takes effect, even if
19544-
`fast_forward` is set to the same duration, since the target time is recalculated
19545-
from the current timestamp on every call.
19537+
description: "The time-skipping configuration for this workflow execution.\n When `fast_forward` is set, time will be fast-forwarded to a future point relative\n to the current workflow timestamp. Each call takes effect, even if\n `fast_forward` is set to the same duration, since the target time is recalculated\n from the current timestamp on every call.\n\n This field must be updated as a whole; updating individual sub-fields is not supported.\n When setting the update mask in `UpdateWorkflowExecutionOptionsRequest`, \n `BatchOperationUpdateWorkflowExecutionOptions`, etc., use a mask that covers the entire field."
1954619538
WorkflowExecutionOptionsUpdatedEventAttributes:
1954719539
type: object
1954819540
properties:
@@ -19577,7 +19569,12 @@ components:
1957719569
timeSkippingConfig:
1957819570
allOf:
1957919571
- $ref: '#/components/schemas/TimeSkippingConfig'
19580-
description: If set, the time-skipping configuration was changed. Contains the full updated configuration.
19572+
description: TimeSkippingConfig override upserted in this event. Represents the full config.
19573+
timeSkippingConfigUpdated:
19574+
type: boolean
19575+
description: |-
19576+
Indicates the time skipping config was updated by the recent call to update
19577+
workflow execution options.
1958119578
workflowUpdateOptions:
1958219579
type: array
1958319580
items:

temporal/api/history/v1/message.proto

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -915,8 +915,14 @@ message WorkflowExecutionOptionsUpdatedEventAttributes {
915915
// Priority override upserted in this event. Represents the full priority; not just partial fields.
916916
// Ignored if nil.
917917
temporal.api.common.v1.Priority priority = 6;
918-
// If set, the time-skipping configuration was changed. Contains the full updated configuration.
918+
919+
// TimeSkippingConfig override upserted in this event. Represents the full config.
919920
temporal.api.common.v1.TimeSkippingConfig time_skipping_config = 7;
921+
922+
// Indicates the time skipping config was updated by the recent call to update
923+
// workflow execution options.
924+
bool time_skipping_config_updated = 9;
925+
920926
// Updates to workflow updates options.
921927
repeated WorkflowUpdateOptionsUpdate workflow_update_options = 8;
922928
}

temporal/api/workflow/v1/message.proto

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -585,14 +585,15 @@ message WorkflowExecutionOptions {
585585
// If set, overrides the workflow's priority sent by the SDK.
586586
temporal.api.common.v1.Priority priority = 2;
587587

588-
// Time-skipping configuration for this workflow execution.
589-
// If not set, the time-skipping configuration is not updated by this request;
590-
// the existing configuration is preserved.
591-
//
588+
// The time-skipping configuration for this workflow execution.
592589
// When `fast_forward` is set, time will be fast-forwarded to a future point relative
593590
// to the current workflow timestamp. Each call takes effect, even if
594591
// `fast_forward` is set to the same duration, since the target time is recalculated
595592
// from the current timestamp on every call.
593+
//
594+
// This field must be updated as a whole; updating individual sub-fields is not supported.
595+
// When setting the update mask in `UpdateWorkflowExecutionOptionsRequest`,
596+
// `BatchOperationUpdateWorkflowExecutionOptions`, etc., use a mask that covers the entire field.
596597
temporal.api.common.v1.TimeSkippingConfig time_skipping_config = 3;
597598
}
598599

0 commit comments

Comments
 (0)