mbox series

[0/3] Fix db-murdering API queries

Message ID 20200317135916.13691-1-dja@axtens.net
Headers show
Series Fix db-murdering API queries | expand

Message

Daniel Axtens March 17, 2020, 1:59 p.m. UTC 
Konstantin observed really poor behaviour of the API when pretty basic
filters are applied.

This will be properly fixed with the new data model, but for now it can be
largely mitigated.

I will backport these to stable/2.1 so anyone sticking on 2.1 for the next
little while can still benefit.

Comments

Stephen Finucane March 19, 2020, 12:04 p.m. UTC | #1 
On Wed, 2020-03-18 at 00:59 +1100, Daniel Axtens wrote:
> Konstantin observed really poor behaviour of the API when pretty basic
> filters are applied.
> 
> This will be properly fixed with the new data model, but for now it can be
> largely mitigated.

Reviewed and applied all these.

> I will backport these to stable/2.1 so anyone sticking on 2.1 for the next
> little while can still benefit.

I'll let you do this. You'll probably want to cut v2.1.6 immediately
afterwards.

Time to cut v2.2.0 too? Reminder that I have the patches necessary for
this proposed already [1]. These just need to be applied. I could
probably do that this weekend seeing as I've nothing else to do
nowadays \o/

Stephen

[1] https://patchwork.ozlabs.org/project/patchwork/list/?series=130013
Daniel Axtens March 19, 2020, 1:06 p.m. UTC | #2 
Stephen Finucane <stephen@that.guru> writes:

> On Wed, 2020-03-18 at 00:59 +1100, Daniel Axtens wrote:
>> Konstantin observed really poor behaviour of the API when pretty basic
>> filters are applied.
>> 
>> This will be properly fixed with the new data model, but for now it can be
>> largely mitigated.
>
> Reviewed and applied all these.
>
>> I will backport these to stable/2.1 so anyone sticking on 2.1 for the next
>> little while can still benefit.
>
> I'll let you do this. You'll probably want to cut v2.1.6 immediately
> afterwards.

Will do. I agree re 2.1.6, I was already planning to do that.

>
> Time to cut v2.2.0 too? Reminder that I have the patches necessary for
> this proposed already [1]. These just need to be applied. I could
> probably do that this weekend seeing as I've nothing else to do
> nowadays \o/

I had a couple of minor tweaks I wanted to make, the biggest one was
filtering new events out of the event log. There was something else too
- but it was in an emacs buffer that I hadn't saved and then my desktop
environment crashed, so hopefully it wasn't too important!

I _hope_ to get that done by this weekend, otherwise I'm quite confident
I can have it done by next weekend.

 - d

>
> Stephen
>
> [1] https://patchwork.ozlabs.org/project/patchwork/list/?series=130013