On 2016-10-12 23:26, Daniel Axtens wrote:
Hi Stephen,

This doesn't have my fix for reducing DB queries:

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.


[1] https://lists.ozlabs.org/pipermail/patchwork/2016-October/003511.html
Patchwork mailing list

Reply via email to