-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Possible memory leak #18
Comments
@gperrow it’s worth noting that this driver is linked from the official django documentation. |
cc @gperrow-SAP |
My apologies for the delay in responding and for the neglect of the Django driver. There's been some rearranging of teams here and this driver seems to have gotten lost in the shuffle. I will make sure these issues get addressed. |
Hi @gperrow-SAP, any news on this? |
1 similar comment
Hi @gperrow-SAP, any news on this? |
@lucaippo @WaYdotNET @gperrow-SAP |
Hi,
we noticed a memory leak on some of our applications using sqlany-django.
After some research we found that the number of instances of
sqlanydb.Root
was steadily increasing at each request.sqlanydb.Root
has indeed his desctructor registered within atexit (this should prevent correct garbage-collection), butsqlanydb
only uses it "class-level cached" insideConnection
, whilesqlany-django
instantiates it directly.Are you aware of this behaviour or am I missing something?
The problem can be reproduced using latest version of sqlany-django (1.13) and Django 1.8, and repeatedly calling a view that uses
sqlany_django
database backend.Either using
django.db.models.Model
ordjango.db.connection
to interact with Sybase will do: memory used by the Django server process should increase by some Kbs per request, going up to several hundreds Mbs.While we are at it, do you still actively support this project? It seems that latest issues and PRs are not getting much attention, and latest Django version supported is 1.8, which support ended April 1 2018.
The text was updated successfully, but these errors were encountered: