Skip to content

Add published Temporal cancellation and cleanup comparison - #149

Merged
rmcdaniel merged 1 commit into
mainfrom
test/temporal-cancellation-comparison
Oct 5, 2026
Merged

rmcdaniel merged 1 commit into
mainfrom
test/temporal-cancellation-comparison

Conversation

@rmcdaniel

Copy link
Copy Markdown
Member

Purpose

Add the reproducible published Temporal comparison needed by #136. It exercises the documented heartbeat distinction using explicit waiting policies and shielded cleanup, and keeps physical callback exit separate from durable cancellation.

Qualification

The exact local Compose fixture passes 18 cases / 36 CANCELED parent-child workflows, three repetitions of six profiles. Both SIGKILL/replacement and another cancellation during the committed root cleanup timer preserve one original request, original reason and timer, and finish cleanup. All six cleanup controls reject stale activity completion while root cleanup is still running.

Async remote activities stop with application heartbeats. Without heartbeats they run for twelve seconds and execute a late side effect. Async local activities stop without heartbeats. Blocking local threads stop after their sleep without executing the late effect. These are supported Python SDK distinctions, not a general product ranking or a throughput benchmark.

Versions, image digests, the verified published CLI archive, explicit configuration, commands and assertions are in the fixture. Actions retains the raw canonical histories, process markers, package hashes and variation as ordinary artifacts, then removes the fixture. The runtime uses the CLI's SQLite development Server and no host ports. A hosted result is required before merge.

Part of #136. This does not close it or qualify a DW published tuple.

@rmcdaniel
rmcdaniel marked this pull request as ready for review October 5, 2026 21:59
@rmcdaniel
rmcdaniel merged commit 2e352ea into main Oct 5, 2026
3 checks passed
@rmcdaniel
rmcdaniel deleted the test/temporal-cancellation-comparison branch October 5, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants