Skip to content

Commit 314ad0a

Browse files
committed
Fix incorrect statement about deleting reserved jobs
Deleting a reserved job does not terminate the running worker - it only removes the reservation record. The worker continues its make() call. The actual risk is duplicated work if the job is refreshed and picked up by another worker.
1 parent 86e21f4 commit 314ad0a

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

docs/src/design/autopopulate-2.0-spec.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -489,7 +489,7 @@ MyTable.jobs.reserved.delete()
489489
MyTable.jobs.refresh()
490490
```
491491

492-
**Important**: Be careful when deleting reserved jobs—you may accidentally terminate jobs that are still running. Coordinate with your orchestration system to identify truly orphaned jobs.
492+
**Note**: Deleting a reserved job does not terminate the running worker—it simply removes the reservation record. If the worker is still running, it will complete its `make()` call. If the job is then refreshed as pending and picked up by another worker, duplicated work may occur. Coordinate with your orchestration system to identify truly orphaned jobs before clearing them.
493493

494494
## Configuration Options
495495

@@ -666,7 +666,7 @@ This section identifies potential hazards and their mitigations.
666666
| Hazard | Description | Mitigation |
667667
|--------|-------------|------------|
668668
| **Accidental job deletion** | User runs `jobs.delete()` without restriction | `delete()` inherits from `delete_quick()` (no confirmation); users must apply restrictions carefully |
669-
| **Clearing active jobs** | User clears reserved jobs while workers are running | Document warning in Orphaned Job Handling; recommend coordinating with orchestrator |
669+
| **Clearing active jobs** | User clears reserved jobs while workers are still running | May cause duplicated work if job is refreshed and picked up again; coordinate with orchestrator |
670670
| **Priority confusion** | User expects higher number = higher priority | Document clearly: lower values are more urgent (0 = highest priority) |
671671

672672
### Migration

0 commit comments

Comments
 (0)