[mythtv-commits] Ticket #12974: Auto expire has race condition

MythTV noreply at mythtv.org
Thu Feb 2 19:09:15 UTC 2017


#12974: Auto expire has race condition
----------------------------------+------------------------
 Reporter:  amb@…                 |          Owner:
     Type:  Bug Report - General  |         Status:  new
 Priority:  minor                 |      Milestone:  29.0
Component:  MythTV - General      |        Version:  0.28.0
 Severity:  medium                |     Resolution:
 Keywords:                        |  Ticket locked:  0
----------------------------------+------------------------

Comment (by amb@…):

 The approach of re-scanning the list for pending deletes when mythbackend
 is started is certainly one way to do it and much better than my idea of a
 5 minute grace time.

 Removing the deletepending flag from the database altogether is a much
 better solution though.
 1. It simplifies things in general
 2. The rescanning at startup (above) is not needed because the list will
 be rescanned after a short time anyway and if there is no deletepending
 flag then it can't stop them being deleted.
 3. It is only a snapshot - a file in the Deleted recordings group that
 isn't pending for deletion right now might be pending for deletion in 1
 second's time so the list is obsolete as soon as it is created.
 4. The UI should already handle the case of files disappearing from
 recording groups (e.g. deleted from a different frontend) so listing a
 file even while it is being deleted and then having it disappear should be
 OK.
 5. The frontend should already handle the case of files being deleted
 while being watched (e.g. deleted permanently from a different frontend)
 so having them listed and selectable for watching even when about to be
 deleted should be OK.

 I can't see anything wrong with removing deletepending completely that
 isn't already a bug of another type (points 3, 4 and 5 above).

--
Ticket URL: <https://code.mythtv.org/trac/ticket/12974#comment:3>
MythTV <http://www.mythtv.org>
MythTV Media Center


More information about the mythtv-commits mailing list