[PATCH 2/2] api: EventList: Only provide JSON version

Daniel Axtens dja at axtens.net
Mon Mar 12 23:37:37 AEDT 2018


Something is very, very slow in the d-r-f browsable API events renderer.

In my MySQL test (~33k patches), the CPU time to render the events list
is ~11s, and the time taken by SQL queries is only ~3s. If the JSON
renderer is used, that drops to 0.2s for the entire page (because less
CPU is used, and - for some as yet unknown reason - a *very* expensive
db query is dropped.)

In my PostgreSQL test (~100k patches), the results are even more stark:
30s of CPU time and 0.2s of DB time goes to 0.25s for the entire page.

Something is seriously, seriously wrong with whatever d-r-f is doing.
So, simply render the event list as bare JSON for now.

There are a few followups we should do, but this worth doing now -
no-one should be able to DoS a patchwork server by just enumerating the
events!

In particular, we should find out:
 - why postgres and mysql behaviour is so different.
 - what on earth d-r-f is doing that makes rendering the pretty-printed
   version so incredibly slow.

Signed-off-by: Daniel Axtens <dja at axtens.net>
---
 patchwork/api/event.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/patchwork/api/event.py b/patchwork/api/event.py
index 7e04b716af1a..425206443ffe 100644
--- a/patchwork/api/event.py
+++ b/patchwork/api/event.py
@@ -22,6 +22,7 @@ from collections import OrderedDict
 from rest_framework.generics import ListAPIView
 from rest_framework.serializers import ModelSerializer
 from rest_framework.serializers import SerializerMethodField
+from rest_framework.renderers import JSONRenderer
 
 from patchwork.api.embedded import CheckSerializer
 from patchwork.api.embedded import CoverLetterSerializer
@@ -85,9 +86,11 @@ class EventSerializer(ModelSerializer):
         read_only_fields = fields
 
 
+
 class EventList(ListAPIView):
     """List events."""
 
+    renderer_classes = (JSONRenderer, )
     serializer_class = EventSerializer
     filter_class = EventFilter
     page_size_query_param = None  # fixed page size
-- 
2.14.1



More information about the Patchwork mailing list