[mythtv] [PATCH] Split program.{start, end}time into separate date and time columns

David Engel gigem at comcast.net
Fri Mar 19 16:43:50 EST 2004


On Fri, Mar 19, 2004 at 08:43:45AM -0800, Matt Zimmerman wrote:
> > 3. The recordoverride JOIN in AddNewRecords needs to be fixed to also
> >    check the date.
> 
> I have a bad feeling that any mangling I do there is going to defeat the
> indexing for this part of the query, and that we will need to split
> date/time in the other tables in order to get the most out of this.

If it's any consolation, the recordoverrides will probably be redone
before too long so any fix here would only need to be short term.

> Might I suggest a database abstraction layer?  This is only a big change
> because the code to access these columns is spread out across 17 files with
> embedded literal SQL.  Something like this will happen again.

Sure, it would be nice, but I wouldn't hold my breath waiting for it
to happen.

David
-- 
David Engel
gigem at comcast.net


More information about the mythtv-dev mailing list