temporalio 0.0.1 → 0.0.2
Sign up to get free protection for your applications and to get access to all the features.
- checksums.yaml +4 -4
- data/README.md +175 -4
- data/bridge/Cargo.lock +44 -21
- data/bridge/Cargo.toml +1 -0
- data/bridge/sdk-core/Cargo.toml +1 -1
- data/bridge/sdk-core/README.md +1 -4
- data/bridge/sdk-core/client/Cargo.toml +1 -1
- data/bridge/sdk-core/client/src/lib.rs +12 -20
- data/bridge/sdk-core/client/src/raw.rs +9 -8
- data/bridge/sdk-core/client/src/retry.rs +100 -23
- data/bridge/sdk-core/core/Cargo.toml +7 -7
- data/bridge/sdk-core/core/benches/workflow_replay.rs +13 -10
- data/bridge/sdk-core/core/src/abstractions.rs +22 -22
- data/bridge/sdk-core/core/src/core_tests/activity_tasks.rs +146 -43
- data/bridge/sdk-core/core/src/core_tests/local_activities.rs +419 -9
- data/bridge/sdk-core/core/src/core_tests/queries.rs +247 -89
- data/bridge/sdk-core/core/src/core_tests/workers.rs +2 -2
- data/bridge/sdk-core/core/src/core_tests/workflow_cancels.rs +1 -1
- data/bridge/sdk-core/core/src/core_tests/workflow_tasks.rs +47 -27
- data/bridge/sdk-core/core/src/lib.rs +139 -32
- data/bridge/sdk-core/core/src/protosext/mod.rs +1 -1
- data/bridge/sdk-core/core/src/replay/mod.rs +185 -41
- data/bridge/sdk-core/core/src/telemetry/log_export.rs +190 -0
- data/bridge/sdk-core/core/src/telemetry/metrics.rs +184 -139
- data/bridge/sdk-core/core/src/telemetry/mod.rs +310 -315
- data/bridge/sdk-core/core/src/telemetry/prometheus_server.rs +4 -3
- data/bridge/sdk-core/core/src/test_help/mod.rs +23 -9
- data/bridge/sdk-core/core/src/worker/activities/local_activities.rs +12 -6
- data/bridge/sdk-core/core/src/worker/activities.rs +40 -23
- data/bridge/sdk-core/core/src/worker/client/mocks.rs +1 -1
- data/bridge/sdk-core/core/src/worker/client.rs +30 -4
- data/bridge/sdk-core/core/src/worker/mod.rs +23 -19
- data/bridge/sdk-core/core/src/worker/workflow/driven_workflow.rs +10 -19
- data/bridge/sdk-core/core/src/worker/workflow/history_update.rs +99 -25
- data/bridge/sdk-core/core/src/worker/workflow/machines/activity_state_machine.rs +1 -5
- data/bridge/sdk-core/core/src/worker/workflow/machines/cancel_external_state_machine.rs +1 -5
- data/bridge/sdk-core/core/src/worker/workflow/machines/cancel_workflow_state_machine.rs +1 -5
- data/bridge/sdk-core/core/src/worker/workflow/machines/child_workflow_state_machine.rs +3 -5
- data/bridge/sdk-core/core/src/worker/workflow/machines/complete_workflow_state_machine.rs +1 -5
- data/bridge/sdk-core/core/src/worker/workflow/machines/continue_as_new_workflow_state_machine.rs +2 -6
- data/bridge/sdk-core/core/src/worker/workflow/machines/fail_workflow_state_machine.rs +1 -5
- data/bridge/sdk-core/core/src/worker/workflow/machines/local_activity_state_machine.rs +24 -22
- data/bridge/sdk-core/core/src/worker/workflow/machines/mod.rs +12 -38
- data/bridge/sdk-core/core/src/worker/workflow/machines/modify_workflow_properties_state_machine.rs +178 -0
- data/bridge/sdk-core/core/src/worker/workflow/machines/patch_state_machine.rs +1 -5
- data/bridge/sdk-core/core/src/worker/workflow/machines/signal_external_state_machine.rs +1 -5
- data/bridge/sdk-core/core/src/worker/workflow/machines/timer_state_machine.rs +1 -5
- data/bridge/sdk-core/core/src/worker/workflow/machines/transition_coverage.rs +8 -2
- data/bridge/sdk-core/core/src/worker/workflow/machines/upsert_search_attributes_state_machine.rs +1 -5
- data/bridge/sdk-core/core/src/worker/workflow/machines/workflow_machines/local_acts.rs +1 -1
- data/bridge/sdk-core/core/src/worker/workflow/machines/workflow_machines.rs +233 -217
- data/bridge/sdk-core/core/src/worker/workflow/machines/workflow_task_state_machine.rs +1 -6
- data/bridge/sdk-core/core/src/worker/workflow/managed_run/managed_wf_test.rs +4 -4
- data/bridge/sdk-core/core/src/worker/workflow/managed_run.rs +13 -5
- data/bridge/sdk-core/core/src/worker/workflow/mod.rs +86 -29
- data/bridge/sdk-core/core/src/worker/workflow/wft_poller.rs +2 -2
- data/bridge/sdk-core/core/src/worker/workflow/workflow_stream.rs +56 -11
- data/bridge/sdk-core/core-api/Cargo.toml +4 -3
- data/bridge/sdk-core/core-api/src/lib.rs +1 -43
- data/bridge/sdk-core/core-api/src/telemetry.rs +147 -0
- data/bridge/sdk-core/core-api/src/worker.rs +13 -0
- data/bridge/sdk-core/etc/deps.svg +115 -140
- data/bridge/sdk-core/etc/regen-depgraph.sh +5 -0
- data/bridge/sdk-core/fsm/rustfsm_procmacro/tests/trybuild/no_handle_conversions_require_into_fail.stderr +1 -1
- data/bridge/sdk-core/histories/evict_while_la_running_no_interference-23_history.bin +0 -0
- data/bridge/sdk-core/histories/evict_while_la_running_no_interference-85_history.bin +0 -0
- data/bridge/sdk-core/protos/api_upstream/buf.yaml +0 -3
- data/bridge/sdk-core/protos/api_upstream/build/go.mod +7 -0
- data/bridge/sdk-core/protos/api_upstream/build/go.sum +5 -0
- data/bridge/sdk-core/protos/api_upstream/{temporal/api/update/v1/message.proto → build/tools.go} +6 -23
- data/bridge/sdk-core/protos/api_upstream/go.mod +6 -0
- data/bridge/sdk-core/protos/api_upstream/temporal/api/batch/v1/message.proto +12 -9
- data/bridge/sdk-core/protos/api_upstream/temporal/api/command/v1/message.proto +20 -19
- data/bridge/sdk-core/protos/api_upstream/temporal/api/common/v1/message.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/batch_operation.proto +3 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/command_type.proto +4 -4
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/common.proto +3 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/event_type.proto +5 -3
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/failed_cause.proto +23 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/{cluster.proto → interaction_type.proto} +10 -11
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/namespace.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/query.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/reset.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/schedule.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/task_queue.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/update.proto +2 -13
- data/bridge/sdk-core/protos/api_upstream/temporal/api/enums/v1/workflow.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/errordetails/v1/message.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/failure/v1/message.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/filter/v1/message.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/history/v1/message.proto +26 -19
- data/bridge/sdk-core/protos/api_upstream/temporal/api/interaction/v1/message.proto +87 -0
- data/bridge/sdk-core/protos/api_upstream/temporal/api/namespace/v1/message.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/operatorservice/v1/request_response.proto +21 -61
- data/bridge/sdk-core/protos/api_upstream/temporal/api/operatorservice/v1/service.proto +2 -21
- data/bridge/sdk-core/protos/api_upstream/temporal/api/query/v1/message.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/replication/v1/message.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/schedule/v1/message.proto +110 -31
- data/bridge/sdk-core/protos/api_upstream/temporal/api/taskqueue/v1/message.proto +4 -4
- data/bridge/sdk-core/protos/api_upstream/temporal/api/version/v1/message.proto +2 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/workflow/v1/message.proto +3 -2
- data/bridge/sdk-core/protos/api_upstream/temporal/api/workflowservice/v1/request_response.proto +60 -16
- data/bridge/sdk-core/protos/api_upstream/temporal/api/workflowservice/v1/service.proto +17 -3
- data/bridge/sdk-core/protos/local/temporal/sdk/core/workflow_activation/workflow_activation.proto +2 -0
- data/bridge/sdk-core/protos/local/temporal/sdk/core/workflow_commands/workflow_commands.proto +8 -1
- data/bridge/sdk-core/protos/testsrv_upstream/temporal/api/testservice/v1/request_response.proto +2 -2
- data/bridge/sdk-core/protos/testsrv_upstream/temporal/api/testservice/v1/service.proto +2 -2
- data/bridge/sdk-core/sdk/Cargo.toml +2 -2
- data/bridge/sdk-core/sdk/src/interceptors.rs +36 -3
- data/bridge/sdk-core/sdk/src/lib.rs +7 -5
- data/bridge/sdk-core/sdk/src/workflow_context.rs +13 -2
- data/bridge/sdk-core/sdk/src/workflow_future.rs +3 -7
- data/bridge/sdk-core/sdk-core-protos/Cargo.toml +1 -1
- data/bridge/sdk-core/sdk-core-protos/build.rs +0 -1
- data/bridge/sdk-core/sdk-core-protos/src/history_builder.rs +65 -18
- data/bridge/sdk-core/sdk-core-protos/src/history_info.rs +22 -22
- data/bridge/sdk-core/sdk-core-protos/src/lib.rs +104 -44
- data/bridge/sdk-core/test-utils/Cargo.toml +2 -1
- data/bridge/sdk-core/test-utils/src/lib.rs +81 -29
- data/bridge/sdk-core/tests/integ_tests/heartbeat_tests.rs +5 -2
- data/bridge/sdk-core/tests/integ_tests/metrics_tests.rs +37 -0
- data/bridge/sdk-core/tests/integ_tests/polling_tests.rs +0 -13
- data/bridge/sdk-core/tests/integ_tests/workflow_tests/local_activities.rs +167 -13
- data/bridge/sdk-core/tests/integ_tests/workflow_tests/modify_wf_properties.rs +53 -0
- data/bridge/sdk-core/tests/integ_tests/workflow_tests/replay.rs +106 -20
- data/bridge/sdk-core/tests/integ_tests/workflow_tests.rs +18 -8
- data/bridge/sdk-core/tests/main.rs +6 -4
- data/bridge/src/connection.rs +81 -62
- data/bridge/src/lib.rs +92 -33
- data/bridge/src/runtime.rs +9 -2
- data/bridge/src/worker.rs +53 -2
- data/lib/bridge.so +0 -0
- data/lib/gen/temporal/api/batch/v1/message_pb.rb +8 -6
- data/lib/gen/temporal/api/command/v1/message_pb.rb +17 -9
- data/lib/gen/temporal/api/common/v1/message_pb.rb +1 -1
- data/lib/gen/temporal/api/enums/v1/batch_operation_pb.rb +2 -1
- data/lib/gen/temporal/api/enums/v1/command_type_pb.rb +3 -1
- data/lib/gen/temporal/api/enums/v1/common_pb.rb +2 -1
- data/lib/gen/temporal/api/enums/v1/event_type_pb.rb +3 -2
- data/lib/gen/temporal/api/enums/v1/failed_cause_pb.rb +7 -1
- data/lib/gen/temporal/api/enums/v1/interaction_type_pb.rb +25 -0
- data/lib/gen/temporal/api/enums/v1/namespace_pb.rb +1 -1
- data/lib/gen/temporal/api/enums/v1/query_pb.rb +1 -1
- data/lib/gen/temporal/api/enums/v1/reset_pb.rb +1 -1
- data/lib/gen/temporal/api/enums/v1/schedule_pb.rb +1 -1
- data/lib/gen/temporal/api/enums/v1/task_queue_pb.rb +1 -1
- data/lib/gen/temporal/api/enums/v1/update_pb.rb +1 -6
- data/lib/gen/temporal/api/enums/v1/workflow_pb.rb +1 -1
- data/lib/gen/temporal/api/errordetails/v1/message_pb.rb +1 -1
- data/lib/gen/temporal/api/failure/v1/message_pb.rb +1 -1
- data/lib/gen/temporal/api/filter/v1/message_pb.rb +1 -1
- data/lib/gen/temporal/api/history/v1/message_pb.rb +19 -18
- data/lib/gen/temporal/api/interaction/v1/message_pb.rb +49 -0
- data/lib/gen/temporal/api/namespace/v1/message_pb.rb +1 -1
- data/lib/gen/temporal/api/operatorservice/v1/request_response_pb.rb +11 -51
- data/lib/gen/temporal/api/operatorservice/v1/service_pb.rb +1 -1
- data/lib/gen/temporal/api/query/v1/message_pb.rb +1 -1
- data/lib/gen/temporal/api/replication/v1/message_pb.rb +1 -1
- data/lib/gen/temporal/api/schedule/v1/message_pb.rb +22 -1
- data/lib/gen/temporal/api/taskqueue/v1/message_pb.rb +2 -2
- data/lib/gen/temporal/api/version/v1/message_pb.rb +1 -1
- data/lib/gen/temporal/api/workflow/v1/message_pb.rb +2 -1
- data/lib/gen/temporal/api/workflowservice/v1/request_response_pb.rb +27 -10
- data/lib/gen/temporal/api/workflowservice/v1/service_pb.rb +1 -1
- data/lib/gen/temporal/sdk/core/workflow_activation/workflow_activation_pb.rb +1 -0
- data/lib/gen/temporal/sdk/core/workflow_commands/workflow_commands_pb.rb +5 -1
- data/lib/temporalio/activity/context.rb +97 -0
- data/lib/temporalio/activity/info.rb +67 -0
- data/lib/temporalio/activity.rb +85 -0
- data/lib/temporalio/bridge/error.rb +8 -0
- data/lib/temporalio/bridge.rb +14 -0
- data/lib/{temporal → temporalio}/client/implementation.rb +49 -48
- data/lib/{temporal → temporalio}/client/workflow_handle.rb +35 -35
- data/lib/{temporal → temporalio}/client.rb +19 -32
- data/lib/{temporal → temporalio}/connection.rb +238 -223
- data/lib/{temporal → temporalio}/data_converter.rb +76 -35
- data/lib/{temporal → temporalio}/error/failure.rb +6 -6
- data/lib/{temporal → temporalio}/error/workflow_failure.rb +4 -2
- data/lib/{temporal → temporalio}/errors.rb +19 -1
- data/lib/{temporal → temporalio}/failure_converter/base.rb +5 -5
- data/lib/{temporal → temporalio}/failure_converter/basic.rb +58 -52
- data/lib/temporalio/failure_converter.rb +7 -0
- data/lib/{temporal → temporalio}/interceptor/chain.rb +2 -1
- data/lib/{temporal → temporalio}/interceptor/client.rb +22 -1
- data/lib/{temporal → temporalio}/payload_codec/base.rb +5 -5
- data/lib/{temporal → temporalio}/payload_converter/base.rb +3 -3
- data/lib/{temporal → temporalio}/payload_converter/bytes.rb +4 -3
- data/lib/{temporal → temporalio}/payload_converter/composite.rb +7 -5
- data/lib/{temporal → temporalio}/payload_converter/encoding_base.rb +4 -4
- data/lib/{temporal → temporalio}/payload_converter/json.rb +4 -3
- data/lib/{temporal → temporalio}/payload_converter/nil.rb +4 -3
- data/lib/temporalio/payload_converter.rb +14 -0
- data/lib/{temporal → temporalio}/retry_policy.rb +4 -4
- data/lib/{temporal → temporalio}/retry_state.rb +1 -1
- data/lib/temporalio/runtime.rb +25 -0
- data/lib/{temporal → temporalio}/timeout_type.rb +2 -2
- data/lib/temporalio/version.rb +3 -0
- data/lib/temporalio/worker/activity_runner.rb +92 -0
- data/lib/temporalio/worker/activity_worker.rb +138 -0
- data/lib/temporalio/worker/reactor.rb +46 -0
- data/lib/temporalio/worker/runner.rb +63 -0
- data/lib/temporalio/worker/sync_worker.rb +88 -0
- data/lib/temporalio/worker/thread_pool_executor.rb +51 -0
- data/lib/temporalio/worker.rb +198 -0
- data/lib/{temporal → temporalio}/workflow/execution_info.rb +4 -4
- data/lib/{temporal → temporalio}/workflow/execution_status.rb +1 -1
- data/lib/{temporal → temporalio}/workflow/id_reuse_policy.rb +6 -6
- data/lib/{temporal → temporalio}/workflow/query_reject_condition.rb +5 -5
- data/lib/temporalio.rb +12 -3
- data/temporalio.gemspec +7 -3
- metadata +79 -56
- data/bridge/sdk-core/bridge-ffi/Cargo.toml +0 -24
- data/bridge/sdk-core/bridge-ffi/LICENSE.txt +0 -23
- data/bridge/sdk-core/bridge-ffi/build.rs +0 -25
- data/bridge/sdk-core/bridge-ffi/include/sdk-core-bridge.h +0 -249
- data/bridge/sdk-core/bridge-ffi/src/lib.rs +0 -825
- data/bridge/sdk-core/bridge-ffi/src/wrappers.rs +0 -211
- data/bridge/sdk-core/core/src/log_export.rs +0 -62
- data/bridge/sdk-core/core/src/worker/workflow/machines/mutable_side_effect_state_machine.rs +0 -127
- data/bridge/sdk-core/core/src/worker/workflow/machines/side_effect_state_machine.rs +0 -71
- data/bridge/sdk-core/protos/api_upstream/temporal/api/cluster/v1/message.proto +0 -83
- data/bridge/sdk-core/protos/local/temporal/sdk/core/bridge/bridge.proto +0 -210
- data/bridge/sdk-core/sdk/src/conversions.rs +0 -8
- data/lib/gen/temporal/api/cluster/v1/message_pb.rb +0 -67
- data/lib/gen/temporal/api/enums/v1/cluster_pb.rb +0 -26
- data/lib/gen/temporal/api/update/v1/message_pb.rb +0 -26
- data/lib/temporal/bridge.rb +0 -14
- data/lib/temporal/failure_converter.rb +0 -8
- data/lib/temporal/payload_converter.rb +0 -14
- data/lib/temporal/runtime.rb +0 -22
- data/lib/temporal/version.rb +0 -3
- data/lib/temporal.rb +0 -8
data/bridge/sdk-core/core/src/worker/workflow/machines/modify_workflow_properties_state_machine.rs
ADDED
@@ -0,0 +1,178 @@
|
|
1
|
+
use super::{
|
2
|
+
workflow_machines::{MachineResponse, WFMachinesError},
|
3
|
+
NewMachineWithCommand,
|
4
|
+
};
|
5
|
+
use crate::worker::workflow::machines::{Cancellable, EventInfo, WFMachinesAdapter};
|
6
|
+
use rustfsm::{fsm, TransitionResult};
|
7
|
+
use temporal_sdk_core_protos::{
|
8
|
+
coresdk::workflow_commands::ModifyWorkflowProperties,
|
9
|
+
temporal::api::{
|
10
|
+
command::v1::Command,
|
11
|
+
enums::v1::{CommandType, EventType},
|
12
|
+
history::v1::HistoryEvent,
|
13
|
+
},
|
14
|
+
};
|
15
|
+
|
16
|
+
fsm! {
|
17
|
+
pub(super) name ModifyWorkflowPropertiesMachine;
|
18
|
+
command ModifyWorkflowPropertiesMachineCommand;
|
19
|
+
error WFMachinesError;
|
20
|
+
shared_state SharedState;
|
21
|
+
|
22
|
+
Created --(CommandScheduled) --> CommandIssued;
|
23
|
+
CommandIssued --(CommandRecorded) --> Done;
|
24
|
+
}
|
25
|
+
|
26
|
+
/// Instantiates an ModifyWorkflowPropertiesMachine and packs it together with the command to
|
27
|
+
/// be sent to server.
|
28
|
+
pub(super) fn modify_workflow_properties(
|
29
|
+
lang_cmd: ModifyWorkflowProperties,
|
30
|
+
) -> NewMachineWithCommand {
|
31
|
+
let sm = ModifyWorkflowPropertiesMachine {
|
32
|
+
state: Created {}.into(),
|
33
|
+
shared_state: (),
|
34
|
+
};
|
35
|
+
let cmd = Command {
|
36
|
+
command_type: CommandType::ModifyWorkflowProperties as i32,
|
37
|
+
attributes: Some(lang_cmd.into()),
|
38
|
+
};
|
39
|
+
NewMachineWithCommand {
|
40
|
+
command: cmd,
|
41
|
+
machine: sm.into(),
|
42
|
+
}
|
43
|
+
}
|
44
|
+
|
45
|
+
type SharedState = ();
|
46
|
+
|
47
|
+
#[derive(Debug, derive_more::Display)]
|
48
|
+
pub(super) enum ModifyWorkflowPropertiesMachineCommand {}
|
49
|
+
|
50
|
+
#[derive(Debug, Default, Clone, derive_more::Display)]
|
51
|
+
pub(super) struct Created {}
|
52
|
+
|
53
|
+
#[derive(Debug, Default, Clone, derive_more::Display)]
|
54
|
+
pub(super) struct CommandIssued {}
|
55
|
+
|
56
|
+
#[derive(Debug, Default, Clone, derive_more::Display)]
|
57
|
+
pub(super) struct Done {}
|
58
|
+
|
59
|
+
impl WFMachinesAdapter for ModifyWorkflowPropertiesMachine {
|
60
|
+
fn adapt_response(
|
61
|
+
&self,
|
62
|
+
_my_command: Self::Command,
|
63
|
+
_event_info: Option<EventInfo>,
|
64
|
+
) -> Result<Vec<MachineResponse>, Self::Error> {
|
65
|
+
Err(Self::Error::Nondeterminism(
|
66
|
+
"ModifyWorkflowProperties does not use state machine commands".to_string(),
|
67
|
+
))
|
68
|
+
}
|
69
|
+
|
70
|
+
fn matches_event(&self, event: &HistoryEvent) -> bool {
|
71
|
+
matches!(event.event_type(), EventType::WorkflowPropertiesModified)
|
72
|
+
}
|
73
|
+
}
|
74
|
+
|
75
|
+
impl Cancellable for ModifyWorkflowPropertiesMachine {}
|
76
|
+
|
77
|
+
impl TryFrom<HistoryEvent> for ModifyWorkflowPropertiesMachineEvents {
|
78
|
+
type Error = WFMachinesError;
|
79
|
+
|
80
|
+
fn try_from(e: HistoryEvent) -> Result<Self, Self::Error> {
|
81
|
+
match e.event_type() {
|
82
|
+
EventType::WorkflowPropertiesModified => {
|
83
|
+
Ok(ModifyWorkflowPropertiesMachineEvents::CommandRecorded)
|
84
|
+
}
|
85
|
+
_ => Err(Self::Error::Nondeterminism(format!(
|
86
|
+
"ModifyWorkflowPropertiesMachine does not handle {e}"
|
87
|
+
))),
|
88
|
+
}
|
89
|
+
}
|
90
|
+
}
|
91
|
+
|
92
|
+
impl TryFrom<CommandType> for ModifyWorkflowPropertiesMachineEvents {
|
93
|
+
type Error = WFMachinesError;
|
94
|
+
|
95
|
+
fn try_from(c: CommandType) -> Result<Self, Self::Error> {
|
96
|
+
match c {
|
97
|
+
CommandType::ModifyWorkflowProperties => {
|
98
|
+
Ok(ModifyWorkflowPropertiesMachineEvents::CommandScheduled)
|
99
|
+
}
|
100
|
+
_ => Err(Self::Error::Nondeterminism(format!(
|
101
|
+
"ModifyWorkflowPropertiesMachine does not handle command type {c:?}"
|
102
|
+
))),
|
103
|
+
}
|
104
|
+
}
|
105
|
+
}
|
106
|
+
|
107
|
+
impl From<CommandIssued> for Done {
|
108
|
+
fn from(_: CommandIssued) -> Self {
|
109
|
+
Self {}
|
110
|
+
}
|
111
|
+
}
|
112
|
+
|
113
|
+
impl From<Created> for CommandIssued {
|
114
|
+
fn from(_: Created) -> Self {
|
115
|
+
Self {}
|
116
|
+
}
|
117
|
+
}
|
118
|
+
|
119
|
+
#[cfg(test)]
|
120
|
+
mod tests {
|
121
|
+
use super::*;
|
122
|
+
use crate::{replay::TestHistoryBuilder, worker::workflow::ManagedWFFunc};
|
123
|
+
use temporal_sdk::{WfContext, WorkflowFunction};
|
124
|
+
use temporal_sdk_core_protos::temporal::api::{
|
125
|
+
command::v1::command::Attributes, common::v1::Payload,
|
126
|
+
};
|
127
|
+
|
128
|
+
#[tokio::test]
|
129
|
+
async fn workflow_modify_props() {
|
130
|
+
let mut t = TestHistoryBuilder::default();
|
131
|
+
t.add_by_type(EventType::WorkflowExecutionStarted);
|
132
|
+
t.add_full_wf_task();
|
133
|
+
t.add_workflow_execution_completed();
|
134
|
+
|
135
|
+
let (k1, k2) = ("foo", "bar");
|
136
|
+
|
137
|
+
let wff = WorkflowFunction::new(move |ctx: WfContext| async move {
|
138
|
+
ctx.upsert_memo([
|
139
|
+
(
|
140
|
+
String::from(k1),
|
141
|
+
Payload {
|
142
|
+
data: vec![0x01],
|
143
|
+
..Default::default()
|
144
|
+
},
|
145
|
+
),
|
146
|
+
(
|
147
|
+
String::from(k2),
|
148
|
+
Payload {
|
149
|
+
data: vec![0x02],
|
150
|
+
..Default::default()
|
151
|
+
},
|
152
|
+
),
|
153
|
+
]);
|
154
|
+
Ok(().into())
|
155
|
+
});
|
156
|
+
let mut wfm = ManagedWFFunc::new(t, wff, vec![]);
|
157
|
+
|
158
|
+
wfm.get_next_activation().await.unwrap();
|
159
|
+
let commands = wfm.get_server_commands().commands;
|
160
|
+
assert!(!commands.is_empty());
|
161
|
+
let cmd = commands[0].clone();
|
162
|
+
assert_eq!(
|
163
|
+
cmd.command_type,
|
164
|
+
CommandType::ModifyWorkflowProperties as i32
|
165
|
+
);
|
166
|
+
assert_matches!(
|
167
|
+
cmd.attributes.unwrap(),
|
168
|
+
Attributes::ModifyWorkflowPropertiesCommandAttributes(msg) => {
|
169
|
+
let fields = &msg.upserted_memo.unwrap().fields;
|
170
|
+
let payload1 = fields.get(k1).unwrap();
|
171
|
+
let payload2 = fields.get(k2).unwrap();
|
172
|
+
assert_eq!(payload1.data[0], 0x01);
|
173
|
+
assert_eq!(payload2.data[0], 0x02);
|
174
|
+
assert_eq!(fields.len(), 2);
|
175
|
+
});
|
176
|
+
wfm.shutdown().await.unwrap();
|
177
|
+
}
|
178
|
+
}
|
@@ -18,7 +18,7 @@
|
|
18
18
|
//! | replaying, no marker | deprecate_patch | Call allowed |
|
19
19
|
|
20
20
|
use super::{
|
21
|
-
workflow_machines::MachineResponse, Cancellable, EventInfo,
|
21
|
+
workflow_machines::MachineResponse, Cancellable, EventInfo, NewMachineWithCommand,
|
22
22
|
OnEventWrapper, WFMachinesAdapter, WFMachinesError,
|
23
23
|
};
|
24
24
|
use crate::protosext::HistoryEventExt;
|
@@ -186,10 +186,6 @@ impl WFMachinesAdapter for PatchMachine {
|
|
186
186
|
fn matches_event(&self, event: &HistoryEvent) -> bool {
|
187
187
|
event.get_patch_marker_details().is_some()
|
188
188
|
}
|
189
|
-
|
190
|
-
fn kind(&self) -> MachineKind {
|
191
|
-
MachineKind::Patch
|
192
|
-
}
|
193
189
|
}
|
194
190
|
|
195
191
|
impl Cancellable for PatchMachine {}
|
@@ -1,5 +1,5 @@
|
|
1
1
|
use super::{
|
2
|
-
workflow_machines::MachineResponse, Cancellable, EventInfo,
|
2
|
+
workflow_machines::MachineResponse, Cancellable, EventInfo, NewMachineWithCommand,
|
3
3
|
OnEventWrapper, WFMachinesAdapter, WFMachinesError,
|
4
4
|
};
|
5
5
|
use rustfsm::{fsm, MachineError, TransitionResult};
|
@@ -268,10 +268,6 @@ impl WFMachinesAdapter for SignalExternalMachine {
|
|
268
268
|
| EventType::SignalExternalWorkflowExecutionFailed
|
269
269
|
)
|
270
270
|
}
|
271
|
-
|
272
|
-
fn kind(&self) -> MachineKind {
|
273
|
-
MachineKind::SignalExternalWorkflow
|
274
|
-
}
|
275
271
|
}
|
276
272
|
|
277
273
|
impl Cancellable for SignalExternalMachine {
|
@@ -2,7 +2,7 @@
|
|
2
2
|
|
3
3
|
use super::{
|
4
4
|
workflow_machines::{MachineResponse, WFMachinesError},
|
5
|
-
Cancellable, EventInfo,
|
5
|
+
Cancellable, EventInfo, NewMachineWithCommand, OnEventWrapper, WFMachinesAdapter,
|
6
6
|
};
|
7
7
|
use rustfsm::{fsm, MachineError, StateMachine, TransitionResult};
|
8
8
|
use std::convert::TryFrom;
|
@@ -246,10 +246,6 @@ impl WFMachinesAdapter for TimerMachine {
|
|
246
246
|
EventType::TimerStarted | EventType::TimerCanceled | EventType::TimerFired
|
247
247
|
)
|
248
248
|
}
|
249
|
-
|
250
|
-
fn kind(&self) -> MachineKind {
|
251
|
-
MachineKind::Timer
|
252
|
-
}
|
253
249
|
}
|
254
250
|
|
255
251
|
impl Cancellable for TimerMachine {
|
@@ -75,8 +75,10 @@ mod machine_coverage_report {
|
|
75
75
|
complete_workflow_state_machine::CompleteWorkflowMachine,
|
76
76
|
continue_as_new_workflow_state_machine::ContinueAsNewWorkflowMachine,
|
77
77
|
fail_workflow_state_machine::FailWorkflowMachine,
|
78
|
-
local_activity_state_machine::LocalActivityMachine,
|
79
|
-
|
78
|
+
local_activity_state_machine::LocalActivityMachine,
|
79
|
+
modify_workflow_properties_state_machine::ModifyWorkflowPropertiesMachine,
|
80
|
+
patch_state_machine::PatchMachine, signal_external_state_machine::SignalExternalMachine,
|
81
|
+
timer_state_machine::TimerMachine,
|
80
82
|
upsert_search_attributes_state_machine::UpsertSearchAttributesMachine,
|
81
83
|
workflow_task_state_machine::WorkflowTaskMachine,
|
82
84
|
};
|
@@ -115,6 +117,7 @@ mod machine_coverage_report {
|
|
115
117
|
let mut cancel_ext = CancelExternalMachine::visualizer().to_owned();
|
116
118
|
let mut la_mach = LocalActivityMachine::visualizer().to_owned();
|
117
119
|
let mut upsert_search_attr = UpsertSearchAttributesMachine::visualizer().to_owned();
|
120
|
+
let mut modify_wf_props = ModifyWorkflowPropertiesMachine::visualizer().to_owned();
|
118
121
|
|
119
122
|
// This isn't at all efficient but doesn't need to be.
|
120
123
|
// Replace transitions in the vizzes with green color if they are covered.
|
@@ -138,6 +141,9 @@ mod machine_coverage_report {
|
|
138
141
|
m @ "UpsertSearchAttributesMachine" => {
|
139
142
|
cover_transitions(m, &mut upsert_search_attr, coverage)
|
140
143
|
}
|
144
|
+
m @ "ModifyWorkflowPropertiesMachine" => {
|
145
|
+
cover_transitions(m, &mut modify_wf_props, coverage)
|
146
|
+
}
|
141
147
|
m => panic!("Unknown machine {}", m),
|
142
148
|
}
|
143
149
|
}
|
data/bridge/sdk-core/core/src/worker/workflow/machines/upsert_search_attributes_state_machine.rs
CHANGED
@@ -2,7 +2,7 @@ use super::{
|
|
2
2
|
workflow_machines::{MachineResponse, WFMachinesError},
|
3
3
|
NewMachineWithCommand,
|
4
4
|
};
|
5
|
-
use crate::worker::workflow::machines::{Cancellable, EventInfo,
|
5
|
+
use crate::worker::workflow::machines::{Cancellable, EventInfo, WFMachinesAdapter};
|
6
6
|
use rustfsm::{fsm, TransitionResult};
|
7
7
|
use temporal_sdk_core_protos::{
|
8
8
|
coresdk::workflow_commands::UpsertWorkflowSearchAttributes,
|
@@ -102,10 +102,6 @@ impl WFMachinesAdapter for UpsertSearchAttributesMachine {
|
|
102
102
|
EventType::UpsertWorkflowSearchAttributes
|
103
103
|
)
|
104
104
|
}
|
105
|
-
|
106
|
-
fn kind(&self) -> MachineKind {
|
107
|
-
MachineKind::UpsertSearchAttributes
|
108
|
-
}
|
109
105
|
}
|
110
106
|
|
111
107
|
impl Cancellable for UpsertSearchAttributesMachine {}
|
@@ -53,7 +53,7 @@ impl LocalActivityData {
|
|
53
53
|
.chain(self.new_requests.drain(..).map(|sa| {
|
54
54
|
self.executing.insert(sa.seq);
|
55
55
|
LocalActRequest::New(NewLocalAct {
|
56
|
-
schedule_time:
|
56
|
+
schedule_time: SystemTime::now(),
|
57
57
|
schedule_cmd: sa,
|
58
58
|
workflow_type: wf_type.to_string(),
|
59
59
|
workflow_exec_info: WorkflowExecution {
|