Dockery/Pinry
1
0
Fork 0
mirror of https://github.com/pinry/pinry.git synced 2025-11-14 00:55:43 +01:00
Code Issues Packages Projects Releases Activity

Loanding pins does no N+1 queries anymore

Browse Source 
This saves another three queries per pin, bringing to total query count
down to 8.

See issue #85 .
This commit is contained in:
Jens Gutermuth
2015-03-23 02:22:35 +01:00
parent 60cf2dcbd7
commit 3bcc6d3152
1 changed files with 6 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
pinry/core/api.py
View File

@@ -1,3 +1,4 @@
from django.core.exceptions import ObjectDoesNotExist
from tastypie import fields from tastypie import fields
from tastypie.authorization import DjangoAuthorization from tastypie.authorization import DjangoAuthorization
from tastypie.constants import ALL, ALL_WITH_RELATIONS from tastypie.constants import ALL, ALL_WITH_RELATIONS
@@ -59,7 +60,10 @@ class UserResource(ModelResource):
def filter_generator_for(size): def filter_generator_for(size):
def wrapped_func(bundle, **kwargs): def wrapped_func(bundle, **kwargs):
return bundle.obj.get_by_size(size) for thumbnail in bundle.obj._prefetched_objects_cache['thumbnail']:
if thumbnail.size == size:
return thumbnail
raise DoesNotExist()
return wrapped_func return wrapped_func
@@ -136,7 +140,7 @@ class PinResource(ModelResource):
'submitter': ALL_WITH_RELATIONS 'submitter': ALL_WITH_RELATIONS
} }
queryset = Pin.objects.all().select_related('submitter'). \ queryset = Pin.objects.all().select_related('submitter'). \
prefetch_related('image', 'tags') prefetch_related('image__thumbnail_set', 'tags')
resource_name = 'pin' resource_name = 'pin'
include_resource_uri = False include_resource_uri = False
always_return_data = True always_return_data = True