Skip to content

SAA drop type and task queue #959

Merged
dandavison merged 1 commit intorelease/v1.6.x-standalone-activityfrom
saa-use-existing
Mar 9, 2026
Merged

SAA drop type and task queue #959
dandavison merged 1 commit intorelease/v1.6.x-standalone-activityfrom
saa-use-existing

Conversation

@dandavison
Copy link
Contributor

@dandavison dandavison commented Mar 9, 2026

What was changed

Drop execution type and task queue from start standalone activity output

Why?

They are not currently guaranteed correct in the conflictPolicy=UseExisting case

Testing

$ temporal activity start --start-to-close-timeout 1s --task-queue hello-standalone-activity-task-queue --type compose_greeting --activity-id act-1 --input '{"greeting": "mygreeting", "name": "myname"}'
Running execution:
  ActivityId  act-1
  RunId       019cd407-83cc-7c8e-ba17-5a3125e4e7cd
  Namespace   default
$ temporal activity result -a act-1
Results:
  Status  COMPLETED
  Result  "mygreeting, myname!"
$ temporal activity start --start-to-close-timeout 1s --task-queue hello-standalone-activity-task-queue --type compose_greeting --activity-id act-2 -o json --input '{"greeting": "mygreeting", "name": "myname"}'
{
  "activityId": "act-2",
  "runId": "019cd408-38a3-7e15-8b36-58f966f156de",
  "namespace": "default"
}

These fields are not reliably available when conflictPolicy=UseExisting
since the server may return an existing execution with different values
than what was requested. Remove them for consistent output; they remain
available via `activity describe`.
@dandavison dandavison requested review from a team as code owners March 9, 2026 18:49
@dandavison dandavison changed the base branch from main to release/v1.6.x-standalone-activity March 9, 2026 18:53
@dandavison dandavison merged commit ec36926 into release/v1.6.x-standalone-activity Mar 9, 2026
3 checks passed
@dandavison dandavison deleted the saa-use-existing branch March 9, 2026 20:13
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