Standalone Activity by dandavison · Pull Request #688 · temporalio/api

dandavison · 2025-12-10T12:26:36Z

Final Standalone Activity API

cretz · 2025-12-10T13:23:57Z

temporal/api/activity/v1/message.proto

+    // Updated on terminal status.
+    int64 state_transition_count = 9;
+    // Updated once on scheduled and once on terminal status.
+    int64 state_size_bytes = 10;


This is missing on the describe result. We should either remove it here or add it there IMO, users shouldn't have to perform a list of one to get this data.

Agree, we will write a script to verify the fields align.

Added to ActivityExecutionInfo

cretz · 2025-12-10T13:24:23Z

temporal/api/activity/v1/message.proto

+    int64 state_size_bytes = 10;
+    // The difference between close time and scheduled time.
+    // This field is only populated if the activity is closed.
+    google.protobuf.Duration execution_duration = 11;


FWIW, I'd be ok if this helper field was removed since it is so easily derived (but no big deal keeping it)

It's a system search attribute that is available for all archetypes. We have it available to query by so I think it makes sense to return it in the list response.

cretz · 2025-12-10T13:25:04Z

temporal/api/enums/v1/activity.proto

+// TODO: Should this be a common execution status? Seems like the other archetypes will share this status.
+// (-- api-linter: core::0216::synonyms=disabled
+//     aip.dev/not-precedent: Named consistently with WorkflowExecutionStatus. --)
+enum ActivityExecutionStatus {


Workflow is considering pause a status, should activity not do the same?

The first release of standalone activity will not have a pause/unpause API.

My understanding was there was an expectation of list working across standalone and workflow activities at some point (hopefully soon). I wonder if we should make this enum general purpose and not specific to standalone, even if standalone can't set it yet.

We already reuse PendingActivityState in both standalone and workflow activities.

We will add the PAUSED execution status when we implement standalone activity pause.

We will add the PAUSED execution status when we implement standalone activity pause.

Can I get confirmation that we will also add it when we implement non-standalone activity returning from list if we do that before standalone activity pause?

cretz · 2025-12-10T13:26:28Z

temporal/api/enums/v1/activity.proto

+// Note that it is *never* valid to have two running instances of the same activity ID.
+//
+// See `ActivityIdReusePolicy` for handling activity ID duplication with a *closed* activity.
+enum ActivityIdConflictPolicy {


Do we not support TERMINATE_EXISTING? Is it something we plan to add in the future?

Not at the moment because of some technical limitations with state based replication but theoretically it is something we should support IMHO. @dandavison can you track this?

cretz · 2025-12-10T13:29:25Z

temporal/api/workflowservice/v1/request_response.proto

+    // A unique identifier for this start request. Typically UUIDv4.
+    string request_id = 3;
+
+    string activity_id = 4;


Don't need to change anything here, but can you confirm server side requires this to be set?

Add a comment saying that this field is required and explain why it should be set and the uniqueness guarantees within the namespace?

Done.

// Identifier for this activity. Required. This identifier should be meaningful in the user's // own system. It must be unique among activities in the same namespace, subject to the rules // imposed by id_reuse_policy and id_conflict_policy.

bergundy · 2025-12-10T23:12:58Z

temporal/api/activity/v1/message.proto

+    // Incremented each time a new attempt is started.
+    // TODO(dandavison): Confirm if this is on scheduled or started.


Suggested change

// Incremented each time a new attempt is started.

// TODO(dandavison): Confirm if this is on scheduled or started.

// Incremented each time an attempt is scheduled.

Agreed, just changed that.

bergundy · 2025-12-10T23:13:47Z

temporal/api/common/v1/message.proto

        string job_id = 1;
    }

+    // A link to an activity.


This isn't supported yet, I wouldn't include it.

bergundy · 2025-12-10T23:14:06Z

temporal/api/enums/v1/activity.proto

+// Status of a standalone activity.
+// The status is updated once, when the activity is originally scheduled, and again when the activity reaches a terminal
+// status.
+// TODO: Should this be a common execution status? Seems like the other archetypes will share this status.


Remove this TODO

Suggested change

// TODO: Should this be a common execution status? Seems like the other archetypes will share this status.

bergundy · 2025-12-10T23:18:34Z

temporal/api/workflowservice/v1/service.proto

+    // TODO: Clarify what happens if there are no more allowed retries after the current attempt.
+    //
+    // It returns success if the requested activity is already closed.
+    // TODO: This ^^ is copied from RequestCancelWorkflowExecution, do we want to preserve this behavior?


Can you please resolve these TODOs?

TODOs removed and docstrings edited.

bergundy · 2025-12-10T23:19:03Z

temporal/api/enums/v1/activity.proto

+    // Requesting to cancel an activity does not automatically transition the activity to canceled status. If the
+    // activity has a currently running attempt, the activity will only transition to canceled status if the current
+    // attempt is unsuccessful.
+    // TODO: Clarify what happens if there are no more allowed retries after the current attempt.


Please resolve this TODO.

bergundy · 2025-12-10T23:19:11Z

temporal/api/enums/v1/activity.proto

+    // heartbeats.
+    ACTIVITY_EXECUTION_STATUS_TERMINATED = 5;
+    // The activity has timed out by reaching the specified shedule-to-start or schedule-to-close timeouts.
+    // TODO: Clarify if there are other conditions where the activity can end up in timed out status.


Please resolve this TODO.

bergundy

Approved with comments.

Missing API support for: - [ ] PauseActivity - [ ] UnpauseActivity - [ ] ResetActivity - [ ] UpdateActivityOptions

Introduce a unified method for polling / long-polling an activity execution for status and/or outcome.

Use execution-specific ID reuse & conflict policies instead of introducing shared policies for all CHASM executions.

Two APIs: 1. **Return Info, optionally waiting for next state change before doing so** In addition to Info it returns a long-poll token. Optionally accepts a long-poll token, meaning "wait for next state change and then return latest info". Optionally includes outcome and input in response. 1. **Long-poll for outcome**

NOTE the base branch is **not master**, it's standalone-activity.

**What changed?** - Removed all references to `ActivityOptions` - Add request_id to `TerminateActivityExecution` - Add `close_time` to `ActivityInfo` --------- Co-authored-by: Fred Tzeng <41805201+fretz12@users.noreply.github.com> Co-authored-by: Dan Davison <dandavison7@gmail.com>

dandavison requested review from a team as code owners December 10, 2025 12:26

dandavison mentioned this pull request Dec 10, 2025

[DO NOT MERGE] Add standalone activity APIs #640

Closed

4 tasks

dandavison requested review from bergundy, fretz12 and maciejdudko December 10, 2025 12:27

cretz reviewed Dec 10, 2025

View reviewed changes

dandavison changed the title ~~Standalone Activity~~ [DO NOT MERGE] Standalone Activity Dec 10, 2025

bergundy reviewed Dec 10, 2025

View reviewed changes

bergundy approved these changes Dec 10, 2025

View reviewed changes

maciejdudko approved these changes Dec 11, 2025

View reviewed changes

cretz approved these changes Dec 12, 2025

View reviewed changes

dandavison force-pushed the standalone-activity branch 3 times, most recently from 1b7bf7c to 5cab57b Compare December 16, 2025 21:50

bergundy and others added 10 commits December 18, 2025 20:06

Add standalone activity APIs

b9c5ff0

Missing API support for: - [ ] PauseActivity - [ ] UnpauseActivity - [ ] ResetActivity - [ ] UpdateActivityOptions

Describe long poll and some renames

14cdd2e

Clarifying comments

07a076c

Flatten IDs, redo operator commands

35de912

Activity operator APIs

ebb1f5e

Standalone Activity -- unify Describe and GetResult methods (#651)

4b7e0f9

Introduce a unified method for polling / long-polling an activity execution for status and/or outcome.

Standalone activity prototype -- ID reuse/conflict policies (#650)

d938845

Use execution-specific ID reuse & conflict policies instead of introducing shared policies for all CHASM executions.

Remove redundant no_wait field

45b32af

Rename Describe => Poll

2fa63b4

Fix copy/paste errors

2b60801

dandavison and others added 15 commits December 18, 2025 20:06

Rename: ActivityListInfo => ActivityExecutionListInfo

eaecc42

Add comment about maintaining subset relation

6bf1e04

Revert to include_info

6b1a244

Fix typos

80d3a8c

[Standalone activity] Cleanup for pre-release scope (#674)

b16c5d9

NOTE the base branch is **not master**, it's standalone-activity.

Remove workflow_id from ActivityExecutionListInfo (#679)

8896a9e

Address review comments from #675

81213c4

Actually remove activity options from describe response

932d03d

Use shared ActivityExecutionOutcome message

63806f3

Rename: PollActivityExecution

b538504

Add state_size_bytes

2606fdf

Resolve TODO

53e8c4f

Polish / code review suggestions

47e7e23

dandavison force-pushed the standalone-activity branch from 5cab57b to 47e7e23 Compare December 19, 2025 01:06

dandavison changed the title ~~[DO NOT MERGE] Standalone Activity~~ Standalone Activity Dec 19, 2025

dandavison merged commit 1a1e74e into master Dec 19, 2025
4 checks passed

dandavison deleted the standalone-activity branch December 19, 2025 20:57

		// Incremented each time a new attempt is started.
		// TODO(dandavison): Confirm if this is on scheduled or started.

	// Incremented each time a new attempt is started.
	// TODO(dandavison): Confirm if this is on scheduled or started.
	// Incremented each time an attempt is scheduled.

Conversation

dandavison commented Dec 10, 2025

Uh oh!

cretz Dec 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cretz Dec 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

cretz Dec 10, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bergundy left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

cretz Dec 10, 2025 •

edited

Loading

cretz Dec 10, 2025 •

edited

Loading

cretz Dec 10, 2025 •

edited

Loading