[PATCH 10/11] Be sensible computing project patch counts

Stewart Smith stewart at linux.ibm.com
Mon Sep 10 12:55:13 AEST 2018


Stephen Finucane <stephen at that.guru> writes:
> On Fri, 2018-08-10 at 18:01 +1000, Stewart Smith wrote:
>> Django actively fights constructing a query that isn't insane.
>> 
>> So, let's go and just execute a raw one. This is all very standard
>> SQL so should execute everywhere without a problem.
>> 
>> With the dataset of patchwork.ozlabs.org, looking at the /project/
>> page for qemu-devel would take 13 queries and 1500ms,
>> with this patch it's down to 11 queries in ~250ms.
>> For the dataset of the netdev list, it's down to 440ms from 1500ms.
>> 
>> Signed-off-by: Stewart Smith <stewart at linux.ibm.com>
>
> I'm generally reluctant to dive into SQL unless it's absolutely
> required. This is simple enough and the gains are significant so

(Imagine all my exasperation at wanting to do that for all the other
patches :)

> Reviewed-by: Stephen Finucane <stephen at that.guru>
>
> Is this a potential bug report for Django?

It probably is. With some luck I'll find the time to write up a small
test case and send it upstream.

-- 
Stewart Smith
OPAL Architect, IBM.



More information about the Patchwork mailing list