Re: PATCH: Window Function support for PostgreSQL 8.4 - Mailing list pgadmin-hackers

From Dave Page
Subject Re: PATCH: Window Function support for PostgreSQL 8.4
Date
Msg-id 937d27e10901120557u15ac6099j9a39375e4be36820@mail.gmail.com
Whole thread Raw
In response to PATCH: Window Function support for PostgreSQL 8.4  (Ashesh Vashi <ashesh.vashi@enterprisedb.com>)
List pgadmin-hackers
On Mon, Jan 12, 2009 at 8:56 AM, Ashesh Vashi
<ashesh.vashi@enterprisedb.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

pgadmin-hackers by date:

Previous
From: svn@pgadmin.org
Date:
Subject: 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
Next
From: "Dave Page"
Date:
Subject: Re: PATCH: Limit 100 default in frmEditGrid.cpp