Re: [PATCH] Use parsed subject for mboxes

2018-04-10 Thread Veronika Kabatova
- Original Message - > From: "Stephen Finucane" > To: "Veronika Kabatova" , "Daniel Axtens" > > Cc: patchwork@lists.ozlabs.org > Sent: Monday, April 9, 2018 4:09:29 PM > Subject: Re: [PATCH] Use parsed subject for mboxes > > On Mon, 2018-04-09 at 09:00 -0400, Veronika Kabatova wrote: >

[PATCH] Fix stuff around mbox header changes

2018-04-10 Thread vkabatov
From: Veronika Kabatova Bundle tests got broken after the subject in mbox was changed from the parsed version to the original one because the tests checked for the presence of patch's name in the response. Fixing this turned out to be a bit tricky since the tests check the mbox attachment and HTM

Re: [PATCH] Add comments REST API

2018-04-10 Thread Daniel Axtens
Stephen Finucane writes: > On Mon, 2018-04-09 at 09:29 -0400, Veronika Kabatova wrote: >> - Original Message - >> > From: "Stephen Finucane" >> > To: vkaba...@redhat.com, patchwork@lists.ozlabs.org >> > Sent: Monday, April 9, 2018 3:02:57 PM >> > Subject: Re: [PATCH] Add comments REST AP

Re: [PATCH v2 4/6] models: Migrate event fields to JSON field

2018-04-10 Thread Stephen Finucane
On Mon, 2018-04-09 at 22:02 +0100, Stephen Finucane wrote: > This allows us to remove swathes of ForeignKeys and greatly speed up the > '/events' endpoint. This comes at the disadvantage of preventing us > indexing the embedded fields and would result in different responses if > the serializers eve

Re: [PATCH] Add comments REST API

2018-04-10 Thread Veronika Kabatova
- Original Message - > From: "Daniel Axtens" > To: "Stephen Finucane" , "Veronika Kabatova" > > Cc: patchwork@lists.ozlabs.org > Sent: Tuesday, April 10, 2018 4:56:59 PM > Subject: Re: [PATCH] Add comments REST API > > Stephen Finucane writes: > > > On Mon, 2018-04-09 at 09:29 -0400,

Re: [PATCH v2 4/6] models: Migrate event fields to JSON field

2018-04-10 Thread Daniel Axtens
> diff --git > a/patchwork/migrations/0027_migrate_data_from_event_fields_to_payload.py > b/patchwork/migrations/0027_migrate_data_from_event_fields_to_payload.py > new file mode 100644 > index ..5dca85c5 > --- /dev/null > +++ b/patchwork/migrations/0027_migrate_data_from_event_fields_to

Re: [PATCH v2 4/6] models: Migrate event fields to JSON field

2018-04-10 Thread Stephen Finucane
On Wed, 2018-04-11 at 02:23 +1000, Daniel Axtens wrote: > > diff --git > > a/patchwork/migrations/0027_migrate_data_from_event_fields_to_payload.py > > b/patchwork/migrations/0027_migrate_data_from_event_fields_to_payload.py > > new file mode 100644 > > index ..5dca85c5 > > --- /dev/null

Re: [PATCH 0/4] Add 'Event.payload' field

2018-04-10 Thread Stephen Finucane
On Thu, 2018-03-29 at 02:36 +1100, Daniel Axtens wrote: > Stephen Finucane writes: > > > On Wed, 2018-03-28 at 10:02 +1100, Daniel Axtens wrote: > > > Stephen Finucane writes: > > > > > > Hi Stephen, > > > > > > > The '/event' API endpoint is really slow due to the amount of JOINs > > > > nece