[mythtv] [mythtv-commits] Ticket #1660: database updates in *recorder.cpp cause dropped data

[Chris Pinkham]

* On Tue Feb 06, 2007 at 06:26:07AM -0000, MythTV wrote:
> #1660: database updates in *recorder.cpp cause dropped data
> ----------------------------------------+-----------------------------------
>  Reporter:  ajlill at ajlc.waterloo.on.ca  |        Owner:  cpinkham
>      Type:  defect                      |       Status:  assigned
>  Priority:  minor                       |    Milestone:  0.21    
> Component:  mythtv                      |      Version:  head    
>  Severity:  medium                      |   Resolution:          
> ----------------------------------------+-----------------------------------
> Comment (by cpinkham):
> 
>  Nix the whole async idea.  After talking this over on IRC, we are going to
>  move these DB updates outside of the recorder and into the main TVRec
>  class.  The various *recorder classes will continue to populate the delta
>  map but the TVRec class will be the consumer and will write the data out
>  to the DB.  I'm removing the patch I mentioned in my previous comment.

For all on -commits and -dev, the implementation of this is still up in the
air so it will be a little longer.  We don't want to have to create another
thread that sits there idle 99% of the time, so I'm going to see if we can
put this in the TVRec's main loop without causing any contention.  The
scheduler/recorder conflict on the recorded table is going to go away, which
would solve this issue, but I'd still like to get these DB updates out of
the main capture threads if possible.

For anyone having the problem, feel free to use the last patch uploaded to
this ticket by ajlill, it should solve your problem temporarily until we
can get a solution into SVN.

--
Chris