[mythtv] Changeset 23057 - mythdbcon.cpp

[Jim Stichnoth]

On Sat, Apr 10, 2010 at 8:11 AM, Daniel Kristjansson
<danielk at cuymedia.net> wrote:
> On Sat, 2010-04-10 at 02:17 -0700, Michael Ling wrote:
>> I'm running trunk-24015 right now, and I'm still getting the problem
>> when exiting mythfrontend where it pauses, and prints
>>
>> Error in my_thread_global_end(): 1 threads didn't exit
>>
>> before returning to my shell.
>>
>> I'm not sure I'm reading this right, but wasn't this changeset put in
>> to address Ticket #7836?  I see the kPurgeTimeout value set to 10,
>> then in the very next changeset (23058), this value is set to 3600.
>> Can someone explain this, cause the changeset description for 23058
>> doesn't address it.
>
> Lowering the purge timeout was just to assist in debugging, it makes
> the problem described in 7836 happen more frequently, not less. It was
> accidentally committed in 23057 and subsequently reverted in 23058.
> It should have been reverted in it's own commit to avoid this kind
> of misunderstanding.

Note that #7836 only keeps the problem from happening while the
program is running.  You should still expect to see the 5-second delay
and the error message when exiting the application.  Fixing that would
require either a fix in Qt, or restructuring of the DB connection pool
code so that only one thread ever opens or closes DB connections.

Jim