Skip to content

fix: separate durable async invoke flow#8636

Merged
seshubaws merged 2 commits intoaws:developfrom
reedham-aws:async-durable-invoke-fix
Feb 6, 2026
Merged

fix: separate durable async invoke flow#8636
seshubaws merged 2 commits intoaws:developfrom
reedham-aws:async-durable-invoke-fix

Conversation

@reedham-aws
Copy link
Contributor

Which issue(s) does this change fix?

There's not an issue, but a bug introduced in #8590.

Why is this change necessary?

Before this change, the "Event" type invoke for durable functions and on-demand functions had been mixed together. The issue is that this results in the durable execution header not being properly returned asynchronously. This broke some of our tests as well as not being the correct experience.

How does it address the issue?

Separate the flows. Durable functions already had its own asynchronous flow that was being preempted by the added one.

What side effects does this change have?

Should only effect durable functions, and should return the flow to the way it was before #8590.

Other note

When testing locally, I found that the sam-durable-function-emulator was still running after a test run. In a subsequent test run, this was causing failures unless I tore the container down manually. I also tested before any of these changes were added and confirmed that's existing behavior. That should probably also be fixed.

Mandatory Checklist

PRs will only be reviewed after checklist is complete

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@reedham-aws reedham-aws requested a review from a team as a code owner February 6, 2026 21:53
@reedham-aws reedham-aws force-pushed the async-durable-invoke-fix branch from 717ee31 to 0f6c32a Compare February 6, 2026 22:07
@seshubaws seshubaws added this pull request to the merge queue Feb 6, 2026
Merged via the queue into aws:develop with commit bf757b9 Feb 6, 2026
44 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants