On Thu, Nov 29, 2012 at 3:03 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> writes:
>> The other interesting question remaining is what to do about the rm_desc
>> function in rmgr.c.  We're split between these two ideas:
>
> Why try to link rmgr.c into frontend versions at all?  Just make
> a new table file that only includes the desc function pointers.
> Yeah, then there would be two table files to maintain, but it's
> not clear to me that it's uglier than these proposals ...

+1.  It's not likely to get updated very often, we can add comments to
remind people to keep them all in sync, and if you manage to screw it
up without noticing then you are adding recovery code that you have
not tested in any way whatsoever.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

In response to

• Re: Enabling frontend-only xlog "desc" routines at 2012-11-29 20:03:48 from Tom Lane

pgsql-hackers by date

Next:	From: Josh Berkus	Date: 2012-11-30 20:10:11
	Subject: Re: Hot Standby Feedback should default to on in 9.3+
Previous:	From: Tom Lane	Date: 2012-11-30 19:59:33
	Subject: Re: Proposal for Allow postgresql.conf values to be changed via SQL