[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.
Stephen
[1]
https://lists.ozlabs.org/pipermail/patchwork/2016-October/003511.html
More information about the Patchwork
mailing list