Too many open DB connections

[Gilles Caulier]

Hum, before to commit, let's me test in my host computer. Applying and
recompiling... Wait and see.

Gilles

2017-12-17 16:59 GMT+01:00 Maik Qualmann <metzpinguin at gmail.com>:

> The patch was not the problem with the SQLite. The error message on the
> face
> recognize (not search) was after a long run that too many threads were
> created. We must clean up the creating threads, this make the new patch.
> Also
> the problem with too many DB connections is fixed. Gilles, what do you
> think
> about commiting the patch?
>
> Maik
>
> Am Sonntag, 17. Dezember 2017, 12:01:17 CET schrieb Gilles Caulier:
> > Hi Maik,
> >
> > You want mean same problem with sqlite ?
> >
> > Yes, we have plenty of bugs about this topic, for ex for Similarity,
> face,
> > and thumbnails databases operation.
> > This mangle the memory and finally, for computer with small memory, crash
> > the application violently.
> > If we can fix this problem, this will be a big step.
> >
> > As my computer has plenty of memory, the problem is difficult to
> reproduce.
> > I need to make a dedicated VM to test your patch.
> >
> > So the problem is not to group transactions to prevent a bloating state
> > with DB ? Fo rex, i know that thumbs registration is very verboze with
> the
> > db and slow down the interface step by step, especially with remote DB.
> > Right ?
> >
> > Anyway, it's fine for me to introduce the patch for 5.9.0.
> >
> > Gilles
> >
> > 2017-12-17 9:46 GMT+01:00 Maik Qualmann <metzpinguin at gmail.com>:
> > > Makes problems with SQLite...
> > >
> > > Maik
> > >
> > > Am Sonntag, 17. Dezember 2017, 09:39:06 CET schrieb Maik Qualmann:
> > > > Hi,
> > > >
> > > > we have a problem with some database operations (MySQL) with too many
> > >
> > > open
> > >
> > > > connections established. For example, recognize faces. Because
> > > > QThreadStorage deletes the thread data when the thread is exit and
> thus
> > > > also closes the DB, the DB is opened again at the next DB action. To
> > > > test
> > > > the patch extensively, I think only to commit for digiKam 5.9.0.
> > > >
> > > > Maik
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/digikam-devel/attachments/20171217/b0929b56/attachment.html>