serializers: Optimize query performance on PatchDetailSerializer for 'patchinterest'

andrepapoti · victor-accarini · commit ee444188f846 · 2025-01-27T17:25:37.000-03:00
Signed-off-by: andrepapoti &lt;andrepapoti@gmail.com&gt;
diff --git a/patchwork/api/patch.py b/patchwork/api/patch.py
@@ -410,6 +410,7 @@ def get_queryset(self):
                 'project',
                 'series__project',
                 'related__patches__project',
+                'patchreviewintention_set__user',
             )
             .select_related('state', 'submitter', 'series')
             .defer('content', 'diff', 'headers')
diff --git a/patchwork/tests/api/test_patch.py b/patchwork/tests/api/test_patch.py
@@ -239,7 +239,7 @@ def test_list_bug_335(self):
         series = create_series()
         create_patches(5, series=series)
 
-        with self.assertNumQueries(5):
+        with self.assertNumQueries(6):
             self.client.get(self.api_url())
 
     @utils.store_samples('patch-detail')

Original file line number	Diff line number	Diff line change
`@@ -410,6 +410,7 @@ def get_queryset(self):`
`410`	`410`	`'project',`
`411`	`411`	`'series__project',`
`412`	`412`	`'related__patches__project',`
	`413`	`+ 'patchreviewintention_set__user',`
`413`	`414`	`)`
`414`	`415`	`.select_related('state', 'submitter', 'series')`
`415`	`416`	`.defer('content', 'diff', 'headers')`