[PATCH v3 3/5] models: Convert Series-Patch relationship to 1:N

Stephen Finucane stephen at that.guru
Mon Oct 15 23:46:53 AEDT 2018


On Mon, 2018-10-15 at 23:43 +1100, Daniel Axtens wrote:
> Stephen Finucane <stephen at that.guru> writes:
> 
> > On Tue, 2018-10-02 at 16:28 +1000, Daniel Axtens wrote:
> > > > > I haven't tried this, I'm just trying to get the migrations straight in
> > > > > my head - both for my own satisfaction and in hopes that anyone who has
> > > > > to apply them manually to a complicated setup will be able to get it right.
> > > > > 
> > > > > Apart from that I think this mostly looks good - I haven't properly
> > > > > looked at the event handling code or the view changes yet but from a
> > > > > quick skim they seem in order. I'll give the series a spin on my
> > > > > instance next.
> > > 
> > > This is still my plan.
> > > 
> > > Thanks,
> > > Daniel
> > 
> > Any updates on this? I'd like to close this out so I can get back to
> > Veronika's improved tagging series. Even if you don't have time, which
> > is fair, I think this is good enough to merge.
> 
> I'm cloning the lkml public-inbox git repos and attempting to import
> them into patchwork (which requires a new helper script). That'll be
> over a million messages. Then I will time the migration and watch the
> memory usage. That's the one thing I want to check; apart from that I
> have no qualms with the series.
> 
> If you want to do this yourself and report back I'm very happy to take
> your word for the results, otherwise it should be no more than a day or
> two given that I've now finished with the URL schema mangling.

If you're able to do it, I'm happy to wait. There's also the Bootstrap
4 thing but that's more subjective and not as crucial to have eyes on
(it's supposed to look almost the same for now).

I'll stick the URL patch on my review queue for sometime this week.

Cheers,
Stephen




More information about the Patchwork mailing list