Skip to content

fix(python): use a queue for ordering durable callbacks - #4821

Open
mrkaye97 wants to merge 10 commits into
mainfrom
mk/durable-callback-ordering-queue
Open

fix(python): use a queue for ordering durable callbacks#4821
mrkaye97 wants to merge 10 commits into
mainfrom
mk/durable-callback-ordering-queue

Conversation

@mrkaye97

Copy link
Copy Markdown
Contributor

Description

Fixes yet another durable callback ordering issue by using a queue to order callbacks so they're fifo in the order the engine returned them (the correct order) to prevent non-determinism errors on replay / restore / etc.

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist

Changes have been:

  • Documented (where applicable)
  • Added to CHANGELOG (where applicable) -- see Keep a Changelog

馃 AI Disclosure
  • I acknowledge that an LLM was used in the creation of this Pull Request, in accordance with Hatchet's AI_POLICY.md.
  • Details: Fable to help diagnose / fix the issue after I wrote the regression tests

@vercel

vercel Bot commented Aug 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
hatchet-docs Ready Ready Preview Aug 27, 2026 9:46pm

Request Review

@github-actions github-actions Bot added documentation Improvements or additions to documentation sdk-py Related to the Python sdk labels Aug 27, 2026

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mrkaye97 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mrkaye97 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mrkaye97 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mrkaye97 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

@greptile-apps greptile-apps Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

mrkaye97 has reached the 50-credit limit for trial accounts. To continue receiving code reviews, upgrade your plan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation sdk-py Related to the Python sdk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant