From: | Greg Stark <greg(dot)stark(at)enterprisedb(dot)com> |
---|---|
To: | Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Is "Window" really safe as a global typedef name? |
Date: | 2008-12-20 08:18:37 |
Message-ID: | 4129BBB5-0AB3-44D2-8125-52B673EA1FCF@enterprisedb.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
OLAPWindow? Gives a clue what meaning of "window" it's referring to...
--
Greg
On 20 Dec 2008, at 03:13, Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com
> wrote:
> Tom Lane wrote:
>> I'm looking at the window-functions patch and wondering just what
>> kind
>> of trouble we'll get into if we leave its new plan node type named
>> just
>> "Window". I've already confirmed that this is a direct conflict
>> against
>> a typedef in <X11/X.h>, and I'd be not the least bit surprised if
>> it's
>> used in even-more-popular system headers on Windows or Darwin. Now
>> maybe you could always get away with not including such headers
>> together
>> with plannodes.h, but it sure looks like problems waiting to happen.
>> So I'm thinking we'd better rename it, but I'm not coming up with
>> anything good; the best I can do after a long day is "EvalWindow",
>> and that doesn't seem particularly inspired. Any suggestions?
>
> EvalWindow sounds like a function in src/backend/executor/.
>
> WindowAgg?
>
> --
> Heikki Linnakangas
> EnterpriseDB http://www.enterprisedb.com
>
> --
> Sent via pgsql-hackers mailing list (pgsql-hackers(at)postgresql(dot)org)
> To make changes to your subscription:
> http://www.postgresql.org/mailpref/pgsql-hackers
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2008-12-20 09:02:39 | Re: Hot standby and b-tree killed items |
Previous Message | Heikki Linnakangas | 2008-12-20 08:13:34 | Re: Is "Window" really safe as a global typedef name? |