fukamachi / cl-dbi

Database independent interface for Common Lisp
202 stars 28 forks source link

- [sqlite driver] mitigated a memory leak. #43

Closed cage2 closed 5 years ago

cage2 commented 5 years ago

Hello!

Thank you for this (as usual) wonderful library ! :)

I think i have spotted a memory leak in sqlite3 driver.

When a lot of queries are performed the memory usage of the system keeps growing.

Seems that this was related to the fact that the driver did not not called "finalize-statement" before. I have added a few of this forms and at least the problem has been reduced a lot.

Tested (and works OK after this patch i.e. no memory leaks) with a million of "SELECTs" on a file database.

Bye! C.

fukamachi commented 5 years ago

Thanks!

cage2 commented 5 years ago

On Fri, Dec 21, 2018 at 12:11:21AM -0800, Eitaro Fukamachi wrote:

Thanks!

You're welcome! :)

And thank you for your excellent work! :)

Bye! C.