Re: Todo: Teach planner to evaluate multiple windows in the optimal order - Mailing list pgsql-hackers

From Ankit Kumar Pandey
Subject Re: Todo: Teach planner to evaluate multiple windows in the optimal order
Date
Msg-id 276ca68b-c21e-b9f8-6ff7-9eaaf21ff015@gmail.com
Whole thread Raw
In response to Re: Todo: Teach planner to evaluate multiple windows in the optimal order  (David Rowley <dgrowleyml@gmail.com>)
List pgsql-hackers
On 07/01/23 07:59, David Rowley wrote:
> On Thu, 5 Jan 2023 at 04:11, Ankit Kumar Pandey <itsankitkp@gmail.com> wrote:
>> Attaching test cases for this (+ small change in doc).
>>
>> Tested this in one of WIP branch where I had modified
>> select_active_windows and it failed
>>
>> as expected.
>>
>> Please let me know if something can be improved in this.
> Thanks for writing that.
>
> I had a look over the patch and ended up making some adjustments to
> the tests. Looking back at 728202b63, I think any tests we add here
> should be kept alongside the tests added by that commit rather than
> tacked on to the end of the test file.  It also makes sense to me just
> to use the same table as the original tests. I also thought the
> comment in select_active_windows should be in the sort comparator
> function instead. I think that's a more likely place to capture the
> attention of anyone making modifications.
Thanks, I will look it through.
> I've now pushed the adjusted patch.
>
I can't seem to find updated patch in the attachment, can you please

forward the patch again.

Thanks.

-- 
Regards,
Ankit Kumar Pandey




pgsql-hackers by date:

Previous
From: Ankit Kumar Pandey
Date:
Subject: Re: Todo: Teach planner to evaluate multiple windows in the optimal order
Next
From: Dean Rasheed
Date:
Subject: Re: add \dpS to psql