[PATCH 05/10] add patch series function

Stephen Finucane stephenfinucane at hotmail.com
Tue Aug 30 09:21:05 AEST 2016


On 23 Aug 17:03, WEN Pingbo wrote:
> Added a ForeignKey in submission, and linked releated patches by
> 'In-Reply-To' or the first 'References'.
> 
> Child emails must fill 'In-Reply-To' or 'References' with correct msg-id,
> or the parent field will keep empty.
> 
> Signed-off-by: WEN Pingbo <wengpingbo at gmail.com>

I need to think on this a little more, but I don't think will scale
very well. The main issues is that it's essentially a linked list,
which SQL doesn't handle well, and the Django ORM handles it even
worse. Some conditions that I think this would fail on:

* The cover letter gets dropped by your mailing list (Mailman?)
    This will mean you've nothing to set as 'parent' for the series
* You're using nested patches (each patch is a reply to the previous
  patch) and one of those gets lost
    Same as above
* You're using nested patches and your series is rather large
    This will require N queries, where N is the number of patches in
    the series

The last of these is probably the most serious.

Have you seen the series work already submitted to the mailing list
[1]? It still needs work, but it's being actively tested and could fit
your purposes once merged.

[1] https://lists.ozlabs.org/pipermail/patchwork/2016-June/002873.html


More information about the Patchwork mailing list