mirrors/django
1
0
Fork 0
mirror of https://github.com/django/django.git synced 2025-10-31 09:41:08 +00:00
Code Issues 32 Releases Wiki Activity

[3.1.x] Fixed #31624 -- Avoided subquery usage on QuerySet.all().delete().

Browse Source 
Thanks Adam Johnson for the report.

Regression in 7acef095d7.

Backport of 972000420e from master
This commit is contained in:
Simon Charette
2020-05-24 09:18:12 -04:00
committed by Mariusz Felisiak
parent 6e3c543909
commit d91095def4
2 changed files with 8 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
django/db/models/sql/compiler.py
View File

@@ -1400,6 +1400,8 @@ class SQLInsertCompiler(SQLCompiler):
class SQLDeleteCompiler(SQLCompiler): class SQLDeleteCompiler(SQLCompiler):
@cached_property @cached_property
def single_alias(self): def single_alias(self):
# Ensure base table is in aliases.
self.query.get_initial_alias()
return sum(self.query.alias_refcount[t] > 0 for t in self.query.alias_map) == 1 return sum(self.query.alias_refcount[t] > 0 for t in self.query.alias_map) == 1
def _as_sql(self, query): def _as_sql(self, query):

6
tests/delete/tests.py
View File

@@ -605,6 +605,12 @@ class DeletionTests(TestCase):
class FastDeleteTests(TestCase): class FastDeleteTests(TestCase):
def test_fast_delete_all(self):
with self.assertNumQueries(1) as ctx:
User.objects.all().delete()
sql = ctx.captured_queries[0]['sql']
# No subqueries is used when performing a full delete.
self.assertNotIn('SELECT', sql)
def test_fast_delete_fk(self): def test_fast_delete_fk(self):
u = User.objects.create( u = User.objects.create(