[PATCH v5 2/7] models: Add 'Series' model and related models

Stephen Finucane stephen at that.guru
Thu Oct 13 20:40:33 AEDT 2016

On 2016-10-12 23:26, Daniel Axtens wrote:
> Hi Stephen,
> This doesn't have my fix for reducing DB queries:
> http://patchwork.ozlabs.org/patch/670292/

Oh, is it needed here? We're not modifying the view code quite yet so I 
didn't think it was. I'll test this evening. I do include a variant of 
it in patch 6 [1], which is different thanks to the move from a 
one-to-many to a many-to-many patch-series relationship and the fact we 
now store cover letter name in SeriesRevision.name.

> If you are respinning, please consider including it. Otherwise it'll
> need to be a follow-up patch - preferably merged at the same time.

Yeah, I think the change in patch 6 covers it, but I can fix patch 2 to 
defer series if it's affecting performance for that commit.



More information about the Patchwork mailing list