Skip to content

prevent scheduling new work on client timeout - #121

Merged
tom-kuchler merged 20 commits into
mainfrom
dev/prevent-scheduling-on-timeout
Sep 7, 2026
Merged

prevent scheduling new work on client timeout#121
tom-kuchler merged 20 commits into
mainfrom
dev/prevent-scheduling-on-timeout

Conversation

@lanzelotm

@lanzelotm lanzelotm commented Jul 1, 2026

Copy link
Copy Markdown
Collaborator

Skip timed-out work before it reaches a compute engine

@lanzelotm
lanzelotm requested a review from tom-kuchler July 16, 2026 17:12
Comment thread multinode/src/client.rs Outdated
Comment thread multinode/src/client.rs Outdated
Comment thread multinode/src/client.rs
Comment thread multinode/src/client.rs Outdated
Comment thread multinode/src/client.rs
Comment thread multinode/src/client.rs

@tom-kuchler tom-kuchler left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Last nitpick.
All the logic part seems sound.

Comment thread multinode/src/client.rs Outdated
tom-kuchler
tom-kuchler previously approved these changes Sep 7, 2026

@tom-kuchler tom-kuchler left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good now, thanks a lot!

@tom-kuchler tom-kuchler left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Fixed the merge conflicts.

@tom-kuchler
tom-kuchler merged commit 0010fb3 into main Sep 7, 2026
8 checks passed
@tom-kuchler
tom-kuchler deleted the dev/prevent-scheduling-on-timeout branch September 7, 2026 09:47
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