On 2016-10-12 23:26, Daniel Axtens wrote:
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 , 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.
Patchwork mailing list