Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend

Поиск
Список
Период
Сортировка
Искать
От
Alvaro Herrera
Тема
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend
Дата
Msg-id
20130108203618.GB9218@alvh.no-ip.org
Ответ на
Список
Дерево обсуждения
[PATCH] xlogreader-v4 Andres Freund <andres@2ndquadrant.com>
[PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Tom Lane <tgl@sss.pgh.pa.us>
Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Tom Lane <tgl@sss.pgh.pa.us>
Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Alvaro Herrera <alvherre@2ndquadrant.com>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Tom Lane <tgl@sss.pgh.pa.us>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Alvaro Herrera <alvherre@2ndquadrant.com>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Alvaro Herrera <alvherre@2ndquadrant.com>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Tom Lane <tgl@sss.pgh.pa.us>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Tom Lane <tgl@sss.pgh.pa.us>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Tom Lane <tgl@sss.pgh.pa.us>
Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common between backend/frontend Alvaro Herrera <alvherre@2ndquadrant.com>
[PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Tom Lane <tgl@sss.pgh.pa.us>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Tom Lane <tgl@sss.pgh.pa.us>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Tom Lane <tgl@sss.pgh.pa.us>
[PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
[PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs Alvaro Herrera <alvherre@2ndquadrant.com>
Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs Magnus Hagander <magnus@hagander.net>
Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs Magnus Hagander <magnus@hagander.net>
Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs "anarazel@anarazel.de" <andres@anarazel.de>
Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs Peter Geoghegan <peter@2ndquadrant.com>
Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs Michael Paquier <michael.paquier@gmail.com>
Re: [PATCH 1/2] Provide a common malloc wrappers and palloc et al. emulation for frontend'ish environs Andres Freund <andres@2ndquadrant.com>
[PATCH 2/2] use pg_malloc instead of an unchecked malloc in pg_resetxlog Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Heikki Linnakangas <hlinnakangas@vmware.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@anarazel.de>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@anarazel.de>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Heikki Linnakangas <hlinnakangas@vmware.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Heikki Linnakangas <hlinnakangas@vmware.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Robert Haas <robertmhaas@gmail.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Heikki Linnakangas <hlinnakangas@vmware.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Steve Singer <steve@ssinger.info>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Steve Singer <steve@ssinger.info>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Andres Freund <andres@2ndquadrant.com>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Steve Singer <steve@ssinger.info>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Alvaro Herrera <alvherre@2ndquadrant.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Tom Lane <tgl@sss.pgh.pa.us>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Heikki Linnakangas <hlinnakangas@vmware.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Heikki Linnakangas <hlinnakangas@vmware.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Robert Haas <robertmhaas@gmail.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Alvaro Herrera <alvherre@2ndquadrant.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Tom Lane <tgl@sss.pgh.pa.us>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Robert Haas <robertmhaas@gmail.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Tom Lane <tgl@sss.pgh.pa.us>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Robert Haas <robertmhaas@gmail.com>
Re: [PATCH 2/5] Make relpathbackend return a statically result instead of palloc()'ing it Noah Misch <noah@leadboat.com>
[PATCH 3/5] Split out xlog reading into its own module called xlogreader Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 3/5] Split out xlog reading into its own module called xlogreader Alvaro Herrera <alvherre@2ndquadrant.com>
[PATCH 4/5] Add pg_xlogdump contrib module Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 4/5] Add pg_xlogdump contrib module Alvaro Herrera <alvherre@2ndquadrant.com>
Re: [PATCH 4/5] Add pg_xlogdump contrib module Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 4/5] Add pg_xlogdump contrib module Alvaro Herrera <alvherre@2ndquadrant.com>
Re: [PATCH 4/5] Add pg_xlogdump contrib module Andres Freund <andres@2ndquadrant.com>
Re: [PATCH 4/5] Add pg_xlogdump contrib module Alvaro Herrera <alvherre@2ndquadrant.com>
[PATCH 5/5] remove spurious space in running_xact's _desc function Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] xlogreader-v4 Andres Freund <andres@2ndquadrant.com>
Re: [PATCH] xlogreader-v4 Thom Brown <thom@linux.com>
Re: [PATCH] xlogreader-v4 Thom Brown <thom@linux.com>
Andres Freund wrote:
> On 2013-01-08 14:35:12 -0500, Tom Lane wrote:
> > Andres Freund  writes:
> > > On 2013-01-08 14:25:06 -0500, Tom Lane wrote:
> > >> This patch seems unnecessary given that we already put a version of Assert()
> > >> into postgres_fe.h.
> > 
> > > The problem is that some (including existing) pieces of code need to
> > > include postgres.h itself, those can't easily include postgres_fe.h as
> > > well without getting into problems with redefinitions.
> > 
> > There is no place, anywhere, that should be including both.  So I don't
> > see the problem.
> 
> Sorry, misremembered the problem somewhat. The problem is that code that
> includes postgres.h atm ends up with ExceptionalCondition() et
> al. declared even if FRONTEND is defined. So if anything uses an assert
> you need to provide wrappers for those which seems nasty. If they are
> provided centrally and check for FRONTEND that problem doesn't exist.

I think the right fix here is to fix things so that postgres.h is not
necessary.  How hard is that?  Maybe it just requires some more
reshuffling of xlog headers.

-- 
Álvaro Herrera                http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services


В списке pgsql-hackers по дате отправления
От: Simon Riggs
Дата:
От: Alvaro Herrera
Дата:
FAQ