Avatar for the workfloworchestrator user
workfloworchestrator
orchestrator-core
BlogDocsChangelog

Improve step timestamping logic

#991Merged
Comparing
464-improve-timestamping
(
7acfc7b
) with
main
(
c628d5c
)
CodSpeed Performance Gauge
0%
Improvements
0
Regressions
0
Untouched
12
New
0
Dropped
0
Ignored
0

Benchmarks

Passed

test_subscription_model_vertical_references
test/unit_tests/domain/test_base_performance.py::test_subscription_model_vertical_references
CodSpeed Performance Gauge
+1%
4.5 ms
4.5 ms
test_subscription_detail_with_domain_model_does_not_exist
test/unit_tests/api/test_subscriptions.py::test_subscription_detail_with_domain_model_does_not_exist
CodSpeed Performance Gauge
0%
4.7 ms
4.7 ms
test_subscription_model_vertical_references_save
test/unit_tests/domain/test_base_performance.py::test_subscription_model_vertical_references_save
CodSpeed Performance Gauge
0%
136 ms
135.6 ms
test_single_complex_subscription
test/unit_tests/graphql/test_subscription.py::test_single_complex_subscription
CodSpeed Performance Gauge
0%
22.1 ms
22.1 ms
test_subscriptions_single_page
test/unit_tests/graphql/test_subscriptions.py::test_subscriptions_single_page
CodSpeed Performance Gauge
0%
96.5 ms
96.4 ms
test_subscription_model_horizontal_references_save
test/unit_tests/domain/test_base_performance.py::test_subscription_model_horizontal_references_save
CodSpeed Performance Gauge
0%
146 ms
145.8 ms
test_single_subscription_with_depends_on_subscriptions
test/unit_tests/graphql/test_subscriptions.py::test_single_subscription_with_depends_on_subscriptions
CodSpeed Performance Gauge
0%
68.7 ms
68.6 ms
test_single_subscription_schema
test/unit_tests/graphql/test_subscriptions.py::test_single_subscription_schema
CodSpeed Performance Gauge
0%
46.9 ms
46.8 ms
test_subscription_detail_with_domain_model
test/unit_tests/api/test_subscriptions.py::test_subscription_detail_with_domain_model
CodSpeed Performance Gauge
0%
20.1 ms
20.1 ms
test_subscription_does_not_exist
test/unit_tests/graphql/test_subscription.py::test_subscription_does_not_exist
CodSpeed Performance Gauge
0%
9.6 ms
9.6 ms
test_single_simple_subscription
test/unit_tests/graphql/test_subscription.py::test_single_simple_subscription
CodSpeed Performance Gauge
0%
10.2 ms
10.2 ms
test_subscription_model_horizontal_references
test/unit_tests/domain/test_base_performance.py::test_subscription_model_horizontal_references
CodSpeed Performance Gauge
0%
9.5 ms
9.5 ms

Commits

Click on a commit to change the comparison range
Base
main
c628d5c
0%
Improve step timestamping logic This commit creates the new started_at field for the process_steps table and renames executed_at to completed_at. Corresponding fields are also created in the REST and GraphQL APIs. The executed_at field in the process_steps table and the "executed" field in the API is left in place for backwards compatibility(but marked deprecated) Step start time is computed immediately before the step is run (in the step decorator's wrapper function), injected into the state, and later removed from the state and used to populate the DB field. A backfill migration is done to populate started at for previously run steps either with the executed time of the previous step, or the start time of the workflow(if it's the first step)
b05d05f
3 days ago
0%
Fixed linting issues
d875f80
3 days ago
0%
Fix migration downgrade; make started_at timestamping work across step types
7d491c6
2 days ago
0%
Add support for start_at for step groups
4dda427
2 days ago
0%
Merge branch 'main' into 464-improve-timestamping
c32ca79
2 days ago
by jbgann
0%
Changes requested in code review
21ddb77
1 day ago
0%
Got process.map working for state updates
7acfc7b
1 day ago
© 2025 CodSpeed Technology
Home Terms Privacy Docs