From: | "Dave Page" <dpage(at)pgadmin(dot)org> |
---|---|
To: | "Ashesh Vashi" <ashesh(dot)vashi(at)enterprisedb(dot)com> |
Cc: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: PATCH: Window Function support for PostgreSQL 8.4 |
Date: | 2009-01-12 13:57:54 |
Message-ID: | 937d27e10901120557u15ac6099j9a39375e4be36820@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgadmin-hackers |
On Mon, Jan 12, 2009 at 8:56 AM, Ashesh Vashi
<ashesh(dot)vashi(at)enterprisedb(dot)com> wrote:
> Hi All,
>
> I have been working on support for WINDOW function in PostgreSQL 8.4.
>
> Please find the patch for it.
Thanks Ashesh. Patch applied with a few minor changes:
- The dialogue really didn't look good with the grid of checkboxes, so
I've split the properties tab into two, one for basic properties and
one for 'options'. It looks much better I think.
- It's not up to us (or even the backend) to decide what languages can
be used to make window functions. Currently it's only C, but that may
change. It makes more sense for the language validators to throw an
error if needed, as noted by Peter Eisentraut on pgsql-hackers justa
few minutes ago.
- Added 'window' as a keyword soit's quoted and highlighted in the STC
as required.
--
Dave Page
EnterpriseDB UK: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Dave Page | 2009-01-12 14:07:51 | Re: PATCH: Limit 100 default in frmEditGrid.cpp |
Previous Message | svn | 2009-01-12 13:53:05 | SVN Commit by dpage: r7548 - in trunk/pgadmin3: . pgadmin/db pgadmin/dlg pgadmin/include/dlg pgadmin/include/parser pgadmin/include/schema pgadmin/schema pgadmin/ui |