Opened 11 years ago

Closed 11 years ago

#5599 closed patch (fixed)

jobqueue: cancel jobs for deleted shows

Reported by: jppoet@… Owned by: Stuart Auchterlonie
Priority: minor Milestone: 0.22
Component: mythtv Version: head
Severity: medium Keywords: jobqueue
Cc: Stuart Auchterlonie Ticket locked: no

Description

When a new job is ready to start, if the show has been "deleted", cancel the job.

To get real adventurous, instead of canceling the job, it could just be put on the end of the queue so it only gets run if no more important jobs are waiting. That is beyond what I am willing to tackle, though.

Attachments (2)

jobqueue-cancel_deleted.patch.bz2 (447 bytes) - added by jppoet@… 11 years ago.
Cancel jobs for deleted shows
jobqueue-ignore_deleted-v2.patch.bz2 (444 bytes) - added by jppoet@… 11 years ago.
update for trunk r18765

Download all attachments as: .zip

Change History (7)

Changed 11 years ago by jppoet@…

Cancel jobs for deleted shows

comment:1 Changed 11 years ago by jppoet@…

I had to bzip the patch because it was being rejected as spam.

comment:2 Changed 11 years ago by jppoet@…

Updated patch to apply against trunk 18765

Changed 11 years ago by jppoet@…

update for trunk r18765

comment:3 Changed 11 years ago by Stuart Auchterlonie

Cc: Stuart Auchterlonie added
Milestone: unknown0.22
Version: unknownhead

Adding myself as a CC cause this issue bugs me, but i've no time to test it for a few weeks.

Stuart

comment:4 Changed 11 years ago by Dibblah

Owner: changed from Isaac Richards to Stuart Auchterlonie
Status: newassigned

assigning is the new 'aide mémoire' that you once looked the wrong way at a ticket.

comment:5 Changed 11 years ago by Stuart Auchterlonie

Resolution: fixed
Status: assignedclosed

(In [19268]) Closes #5599. If a recording is deleted before any queued jobs for that recording are started, then cancel those queued jobs.

One thing it doesn't address is if you delete a recording while the job is in progress then the transcode will error rather than complete with cancelled as the status.

Note: See TracTickets for help on using tickets.