From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
---|---|
To: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
Cc: | Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: palloc unification |
Date: | 2013-02-06 14:40:37 |
Message-ID: | CA+Tgmoa7KtQm+1MSL+4LEZqF4BxoUASdhgTXtMWiZR0m-pSCeQ@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Feb 6, 2013 at 9:38 AM, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> wrote:
> Robert Haas escribió:
>> On Mon, Feb 4, 2013 at 5:50 PM, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> wrote:
>
>> > I propose to have a new subdirectory src/include/shared, and two
>> > header files:
>
>> > The frontend (pg_malloc) function definitions would live somewhere in,
>> > say, src/shared/fe_memutils.c. For the palloc() implementation, I think
>> > we should create another file, so that frontend-only programs that do
>> > not require those symbols (most of them) are not unnecessarily bloated.
>> >
>> > Opinions on having a new subdir?
>
>> I like the idea of having a place for shared frontend and backend code
>> very much, but I don't think src/include/shared or src/shared is a
>> good name, because "shared" can mean a lot of things - like "shared
>> library", for example. I think that this should be set up in a manner
>> analogous to libpgport, except not for portability code, but instead
>> for other stuff. Maybe we could call it libpgframework or something.
>
> Yeah, I am doing this right now and the "shared" name doesn't seem so
> good. "libpgframework" sounds decent. So since libpgport comes from
> src/port, are we okay with src/framework and src/include/framework?
That seems reasonable to me.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
From | Date | Subject | |
---|---|---|---|
Next Message | Alvaro Herrera | 2013-02-06 14:40:38 | Re: sql_drop Event Trigger |
Previous Message | Robert Haas | 2013-02-06 14:39:27 | Re: sql_drop Event Trigger |