From: | Surafel Temesgen <surafel3000(at)gmail(dot)com> |
---|---|
To: | Krasiyan Andreev <krasiyan(at)gmail(dot)com> |
Cc: | Vik Fearing <vik(dot)fearing(at)2ndquadrant(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [PATCH] distinct aggregates within a window function WIP |
Date: | 2020-09-16 08:19:06 |
Message-ID: | CALAY4q_dLxaxTNWa5edOEG_ZNASzSD=bEedURMYyhtsFg5XuJg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Mar 5, 2020 at 4:17 AM Krasiyan Andreev <krasiyan(at)gmail(dot)com> wrote:
> I have currently suspended development of this patch, based on it's
> review,
> but I will continue development of the other Oliver Ford's work about
> adding support of respect/ignore nulls
> for lag(),lead(),first_value(),last_value() and nth_value() and from
> first/last for nth_value() patch,
> but I am not sure how to proceed with it's implementation and any feedback
> will be very helpful.
>
>
* I applied your patch on top of 58c47ccfff20b8c125903 . It applied cleanly
, compiled, make check pass, but it have white space errors:
*Added functions on windowfuncs.c have no comments so it's not easily
understandable.
* Regression test addition seems huge to me. Can you reduce that? You can
use existing tables and fewer records.
* I don’t understand why this patch has to change makeBoolAConst? It
already make “bool” constant node
regards
Surafel
From | Date | Subject | |
---|---|---|---|
Next Message | Heikki Linnakangas | 2020-09-16 08:20:48 | Re: Yet another fast GiST build |
Previous Message | Greg Nancarrow | 2020-09-16 07:50:03 | Re: Parallel copy |