<div dir="ltr">Q : your new patch is smaller than first one. It's normal ? I need to apply both or first is merged in second and improved ?<div><br></div><div>Gilles</div></div><div class="gmail_extra"><br><div class="gmail_quote">2017-12-17 17:02 GMT+01:00 Gilles Caulier <span dir="ltr"><<a href="mailto:caulier.gilles@gmail.com" target="_blank">caulier.gilles@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hum, before to commit, let's me test in my host computer. Applying and recompiling... Wait and see.<span class="HOEnZb"><font color="#888888"><div><br></div><div>Gilles</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">2017-12-17 16:59 GMT+01:00 Maik Qualmann <span dir="ltr"><<a href="mailto:metzpinguin@gmail.com" target="_blank">metzpinguin@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">The patch was not the problem with the SQLite. The error message on the face<br>
recognize (not search) was after a long run that too many threads were<br>
created. We must clean up the creating threads, this make the new patch. Also<br>
the problem with too many DB connections is fixed. Gilles, what do you think<br>
about commiting the patch?<br>
<span class="m_1182523514548672650HOEnZb"><font color="#888888"><br>
Maik<br>
</font></span><div class="m_1182523514548672650HOEnZb"><div class="m_1182523514548672650h5"><br>
Am Sonntag, 17. Dezember 2017, 12:01:17 CET schrieb Gilles Caulier:<br>
> Hi Maik,<br>
><br>
> You want mean same problem with sqlite ?<br>
><br>
> Yes, we have plenty of bugs about this topic, for ex for Similarity, face,<br>
> and thumbnails databases operation.<br>
> This mangle the memory and finally, for computer with small memory, crash<br>
> the application violently.<br>
> If we can fix this problem, this will be a big step.<br>
><br>
> As my computer has plenty of memory, the problem is difficult to reproduce.<br>
> I need to make a dedicated VM to test your patch.<br>
><br>
> So the problem is not to group transactions to prevent a bloating state<br>
> with DB ? Fo rex, i know that thumbs registration is very verboze with the<br>
> db and slow down the interface step by step, especially with remote DB.<br>
> Right ?<br>
><br>
> Anyway, it's fine for me to introduce the patch for 5.9.0.<br>
><br>
> Gilles<br>
><br>
> 2017-12-17 9:46 GMT+01:00 Maik Qualmann <<a href="mailto:metzpinguin@gmail.com" target="_blank">metzpinguin@gmail.com</a>>:<br>
> > Makes problems with SQLite...<br>
> ><br>
> > Maik<br>
> ><br>
> > Am Sonntag, 17. Dezember 2017, 09:39:06 CET schrieb Maik Qualmann:<br>
> > > Hi,<br>
> > ><br>
> > > we have a problem with some database operations (MySQL) with too many<br>
> ><br>
> > open<br>
> ><br>
> > > connections established. For example, recognize faces. Because<br>
> > > QThreadStorage deletes the thread data when the thread is exit and thus<br>
> > > also closes the DB, the DB is opened again at the next DB action. To<br>
> > > test<br>
> > > the patch extensively, I think only to commit for digiKam 5.9.0.<br>
> > ><br>
> > > Maik</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>