In redis scheduler removes items that are queued for too long #1414
Conversation
There was a problem hiding this comment.
Reviewable status: 0 of 1 LGTMs obtained, and 0 of 14 files reviewed, and pending CI: Analyze (javascript-typescript), Analyze (python), Bazel Dev / macos-13, Bazel Dev / ubuntu-24.04, Cargo Dev / ubuntu-22.04, Coverage, Installation / macos-13, Installation / macos-14, Installation / ubuntu-22.04, Local / ubuntu-22.04, NativeLink.com Cloud / Remote Cache / macos-14, NativeLink.com Cloud / Remote Cache / ubuntu-24.04, Publish image, Publish nativelink-worker-init, Publish nativelink-worker-lre-cc, Remote / large-ubuntu-22.04, Web Platform Deployment / macos-14, Web Platform Deployment / ubuntu-24.04, asan / ubuntu-22.04, docker-compose-compiles-nativelink (22.04), integration-tests (22.04), macos-13, pre-commit-checks, ubuntu-20.04 / stable, ubuntu-22.04, ubuntu-22.04 / stable, vale, windows-2022 / stable (waiting on @adam-singer)
In the event a client is no longer requesting to execute a task, the scheduler will now move items out of the queued state after 60 seconds (configurable). This will ensure we don't end up with items in the queue forever.
allada
force-pushed
the
remove-items-queued-for-too-long-in-redis-scheduler
branch
from
deaa4ba to
8b12016
Compare
There was a problem hiding this comment.
Reviewed 7 of 7 files at r1, all commit messages.
Reviewable status: 0 of 1 LGTMs obtained, and all files reviewed (waiting on @adam-singer)
There was a problem hiding this comment.
Reviewable status:
complete! 1 of 1 LGTMs obtained, and all files reviewed
allada
deleted the
remove-items-queued-for-too-long-in-redis-scheduler
branch
In the event a client is no longer requesting to execute a task, the
scheduler will now move items out of the queued state after 60 seconds
(configurable). This will ensure we don't end up with items in the queue
forever.
This change is