-
-
Notifications
You must be signed in to change notification settings - Fork 1k
feat(server): New TTL system, enforce max queue length limits, lazy waitpoint creation #2980
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
ericallam
wants to merge
15
commits into
main
Choose a base branch
from
ea-branch-117
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+6,595
−457
Open
Changes from all commits
Commits
Show all changes
15 commits
Select commit
Hold shift + click to select a range
0326ff1
feat(dashboard): Display environment queue length limits on queues an…
ericallam be8fa57
Make it clear the limit is across all queues in the env
ericallam 4918f83
A couple of devin improvements and adding an in memory cache for the …
ericallam 9aac1c4
Add queue length limits at the queue level, lazy waitpoint creation, …
ericallam e93dbba
Failed batch queue processing now creates a pre-failed run, better ha…
ericallam 45b6cdb
introduce maximum ttl via the RUN_ENGINE_DEFAULT_MAX_TTL optional env…
ericallam 28d955c
fix webapp typecheck issue
ericallam 6004591
improve efficiency of expiring runs in batch, and make sure runs are …
ericallam 5eee2df
Make sure maxTtl is enforced even when the ttl option passed in does …
ericallam 17aabd4
Create a more reliable ttl expiration system using atomic redis worker
ericallam 037826b
queue size limits are upgradable and don't make the max dev queue hav…
ericallam 441386f
correctly clear runs from env current concurrency sets when dequeued …
ericallam d424754
Only engage the ttl system when a run is first enqueued, no longer on…
ericallam 7c143dd
Ensure lazy waitpoints are always created and completed even if the e…
ericallam 471bf19
Adopt the pre-failed run approach in the legacy run engine batch trig…
ericallam File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.