diff options
| author | Simon Charette <charette.s@gmail.com> | 2025-01-19 21:35:50 -0500 |
|---|---|---|
| committer | Sarah Boyce <42296566+sarahboyce@users.noreply.github.com> | 2025-01-21 09:13:18 +0100 |
| commit | 9861e8654701bf8b95b2b533034e78058eee71b4 (patch) | |
| tree | f628b7164e913aed6734539358560073d5747852 /tests/foreign_object | |
| parent | 55afe50aca9fa5f4730a85bf0b6a4807e44c9d35 (diff) | |
[5.2.x] Fixed #36116 -- Optimized multi-column ForwardManyToOne prefetching.
Rely on ColPairs and TupleIn which support a single column to be specified
to avoid special casing ForwardManyToOne.get_prefetch_querysets().
Thanks Jacob Walls for the report.
Backport of 626d77e52a3f247358514bcf51c761283968099c from main.
Diffstat (limited to 'tests/foreign_object')
| -rw-r--r-- | tests/foreign_object/models/person.py | 2 | ||||
| -rw-r--r-- | tests/foreign_object/tests.py | 39 |
2 files changed, 37 insertions, 4 deletions
diff --git a/tests/foreign_object/models/person.py b/tests/foreign_object/models/person.py index f0848e6c3e..d536ab63d7 100644 --- a/tests/foreign_object/models/person.py +++ b/tests/foreign_object/models/person.py @@ -107,6 +107,6 @@ class Friendship(models.Model): Person, from_fields=["to_friend_country_id", "to_friend_id"], to_fields=["person_country_id", "id"], - related_name="to_friend", + related_name="to_friend+", on_delete=models.CASCADE, ) diff --git a/tests/foreign_object/tests.py b/tests/foreign_object/tests.py index e288ecd7d4..8b36df29d7 100644 --- a/tests/foreign_object/tests.py +++ b/tests/foreign_object/tests.py @@ -4,7 +4,7 @@ import pickle from operator import attrgetter from django.core.exceptions import FieldError -from django.db import models +from django.db import connection, models from django.test import SimpleTestCase, TestCase, skipUnlessDBFeature from django.test.utils import isolate_apps from django.utils import translation @@ -247,7 +247,7 @@ class MultiColumnFKTests(TestCase): normal_people = [m.person for m in Membership.objects.order_by("pk")] self.assertEqual(people, normal_people) - def test_prefetch_foreignkey_forward_works(self): + def test_prefetch_foreignobject_forward(self): Membership.objects.create( membership_country=self.usa, person=self.bob, group=self.cia ) @@ -264,7 +264,40 @@ class MultiColumnFKTests(TestCase): normal_people = [m.person for m in Membership.objects.order_by("pk")] self.assertEqual(people, normal_people) - def test_prefetch_foreignkey_reverse_works(self): + def test_prefetch_foreignobject_hidden_forward(self): + Friendship.objects.create( + from_friend_country=self.usa, + from_friend_id=self.bob.id, + to_friend_country_id=self.usa.id, + to_friend_id=self.george.id, + ) + Friendship.objects.create( + from_friend_country=self.usa, + from_friend_id=self.bob.id, + to_friend_country_id=self.soviet_union.id, + to_friend_id=self.sam.id, + ) + with self.assertNumQueries(2) as ctx: + friendships = list( + Friendship.objects.prefetch_related("to_friend").order_by("pk") + ) + prefetch_sql = ctx[-1]["sql"] + # Prefetch queryset should be filtered by all foreign related fields + # to prevent extra rows from being eagerly fetched. + prefetch_where_sql = prefetch_sql.split("WHERE")[-1] + for to_field_name in Friendship.to_friend.field.to_fields: + to_field = Person._meta.get_field(to_field_name) + with self.subTest(to_field=to_field): + self.assertIn( + connection.ops.quote_name(to_field.column), + prefetch_where_sql, + ) + self.assertNotIn(" JOIN ", prefetch_sql) + with self.assertNumQueries(0): + self.assertEqual(friendships[0].to_friend, self.george) + self.assertEqual(friendships[1].to_friend, self.sam) + + def test_prefetch_foreignobject_reverse(self): Membership.objects.create( membership_country=self.usa, person=self.bob, group=self.cia ) |
