Opened 16 years ago
Last modified 10 months ago
#10919 new New feature
Add an option to disable display of related items on admin's delete confirmation page (to prevent large memory usage on complex objects)
Reported by: | Tobias McNulty | Owned by: | nobody |
---|---|---|---|
Component: | contrib.admin | Version: | dev |
Severity: | Normal | Keywords: | admin memory limit |
Cc: | aaron@…, sasha@…, michal@…, Sergii Lapin | Triage Stage: | Accepted |
Has patch: | no | Needs documentation: | no |
Needs tests: | no | Patch needs improvement: | no |
Easy pickings: | no | UI/UX: | no |
Description
I recently tried to delete an object in the admin that had several million related objects.
The server quickly ran out of memory as the apache process's memory usage ballooned upwards to near a gigabyte.
I assume this is because it was trying to create an HTML page listing out all the related objects.
Can the admin page do a count and/or limit to avoid this?
I'm using Django 1.1 trunk (r10628), mod_wsgi 2.0, and apache 2.2.8.
Change History (13)
comment:1 by , 15 years ago
Triage Stage: | Unreviewed → Accepted |
---|
comment:2 by , 14 years ago
Severity: | → Normal |
---|---|
Type: | → Bug |
comment:3 by , 13 years ago
Easy pickings: | unset |
---|---|
UI/UX: | unset |
comment:4 by , 9 years ago
Summary: | admin object deletion confirmation page causes server out of memory error → Add an option to disable display of related items on admin's delete confirmation page (to prevent large memory usage on complex objects) |
---|---|
Type: | Bug → New feature |
Version: | 1.1-beta → master |
An option to disable or limit the display of related objects on the delete confirmation page seems like a workable solution here (other ideas welcome).
comment:5 by , 9 years ago
Cc: | added |
---|
comment:7 by , 8 years ago
Adding such property to ModelAdmin sounds like a good idea.
I've workarounded it myself by removing deleted_objects before rendering the template:
def render_delete_form(self, request, context): context['deleted_objects'] = [_('Object listing disabled')] return super(ProjectAdmin, self).render_delete_form(request, context)
This way I will get the summary (so that user has idea what he is deleting), but not object list as it is too long to get displayed.
comment:8 by , 8 years ago
Cc: | added |
---|
follow-up: 12 comment:9 by , 6 years ago
Actually, removing deleted_objects from context doesn't prevent from collecting related items by django.contrib.admin.utils.NestedObjects in django.contrib.admin.utils.get_deleted_objects
Maybe it better off to override get_deleted_objects function (Django 2.1+ has this method inside AdminModel class). But there appears new issue: How to check permissions and protected foreign keys for related items?..
comment:10 by , 6 years ago
Cc: | added |
---|
comment:11 by , 5 years ago
Replying to Tobias McNulty:
I recently tried to delete an object in the admin that had several million related objects.
The server quickly ran out of memory as the apache process's memory usage ballooned upwards to near a gigabyte.
I assume this is because it was trying to create an HTML page listing out all the related objects.
Can the admin page do a count and/or limit to avoid this?
I'm using Django 1.1 trunk (r10628), mod_wsgi 2.0, and apache 2.2.8.
I tried one method. I don't want related objects. So i overrided the get_deleted_object() method. But i removed "to_delete = collector.nested(format_callback)" line and return just the queryset(objs). So it will show only the objects to delete. Is there any issue with this method? If there any, please tell me.
comment:12 by , 5 years ago
Replying to Tobias McNulty:
I recently tried to delete an object in the admin that had several million related objects.
The server quickly ran out of memory as the apache process's memory usage ballooned upwards to near a gigabyte.
I assume this is because it was trying to create an HTML page listing out all the related objects.
Can the admin page do a count and/or limit to avoid this?
I'm using Django 1.1 trunk (r10628), mod_wsgi 2.0, and apache 2.2.8.
I tried one method. I don't want related objects. So i overrided the get_deleted_object() method. But i removed "to_delete = collector.nested(format_callback)" line and return just the queryset(objs). So it will show only the objects to delete. Is there any issue with this method? If there any, please tell me. I am using djang 1.11.
comment:13 by , 5 years ago
I am using Django 2.2.10, and this is still an issue.
If I am using Django Admin to delete an object with millions of related records, it tries to generate the list, takes too long, and my browser gives up waiting, showing an error loading the page.
I'm not sure how difficult this is to do, but I have some thoughts:
When deleting an object from Django Admin, the summary section is great, but I really like the fact that Django Admin lists all effected related objects. If the summary section was the only thing on the page, someone deleting an object may think "pfft - I know exactly what happens if I delete this", and delete it without a second thought, without even bothering to glance at the summary of the number of related objects that will be effected. I know this sounds like a "that's their problem" type of deal, but if we can prevent this, why wouldn't we?
The list of effected objects that Django Admin currently provides is very clear. As soon as the page loads, if you think you are about to delete one object, but you see a massive list of effected stuff, you immediately start looking into what you are deleting, and why it effects more than what you expected.
I think a good solution would be a middle-ground, rather than disabling the list of related objects.
If you are about to delete an object with a million related objects, have django do a DB query with something like "limit 100". It will pull 100 related objects, quickly generate the HTML page without trying to list a million objects on the page, and the user will still have the advantage of quickly seeing that his delete operation will effect a ton of objects, since he'll have a hundred objects listed on the page.
To make things clear, add a message at the bottom that says something like "and 999,900 others" or something. This will tell them that the 100 objects listed aren't the only ones to be deleted.
Any thoughts?
This bug is still sitting around. Maybe we should document it somewhere, that for complex objects...