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
Ответ на
Re: [PATCH 1/5] Centralize Assert* macros into c.h so its common
between backend/frontend (Andres Freund)
Список
Дерево обсуждения
[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 Tom Lane <tgl@sss.pgh.pa.us>
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 Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
Re: Re: [PATCH] unified frontend support for pg_malloc et al and palloc/pfree mulation (was xlogreader-v4) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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) Tom Lane <tgl@sss.pgh.pa.us>
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
Дата: