[mythtv-commits] Ticket #6478: Add update timestamp to program table for incremental program data retrieval

MythTV mythtv at cvs.mythtv.org
Wed Apr 29 01:02:33 UTC 2009


#6478: Add update timestamp to program table for incremental program data
retrieval
---------------------------------------+------------------------------------
 Reporter:  chasedouglas at gmail.com     |        Owner:  danielk 
     Type:  patch                      |       Status:  assigned
 Priority:  minor                      |    Milestone:  unknown 
Component:  MythTV - Mythfilldatabase  |      Version:  head    
 Severity:  medium                     |   Resolution:          
  Mlocked:  0                          |  
---------------------------------------+------------------------------------

Comment(by anonymous):

 I added one last revision of the patch to fix some sql query issues with
 the datadirect program table update code.

  1. originalairdate may be NULL, and thus the null-safe comparison
 operator (<=>) should be used.
  2. The bit shift for the audioprop comparison was oriented in the wrong
 direction.
  3. A few extra WHERE clauses to deal with situations where the dd_genre
 table does not have any information about a program in dd_v_program.

 After this patch I got one days worth of program table updates down from
 406 changes to just two row changes I can't work around. The two changes
 were due to marking programs as the first part of a multiple part series.
 I can't think of a good way to get around having to update such rows on
 every mythfilldatabase run.

 These last changes were nagging at me because I thought 406/~2000 programs
 per day being updated on each mythfilldatabase run wasn't as good as it
 should be. Now that it's down to just a few programs that are marked
 specially due to their part numbers in a series I believe this patch is
 functionally correct. Only regression testing should need to be done at
 this point.

-- 
Ticket URL: <http://svn.mythtv.org/trac/ticket/6478#comment:3>
MythTV <http://www.mythtv.org/>
MythTV


More information about the mythtv-commits mailing list