Re: exposing pg_controldata and pg_config as functions

On Sun, Jan 17, 2016 at 02:24:46PM -0800, Joe Conway wrote:
> On 01/16/2016 06:02 AM, Michael Paquier wrote:
> > On Wed, Dec 30, 2015 at 9:08 AM, Joe Conway <mail@joeconway.com> wrote:
> >> 1) Change NextXID output format from "%u/%u" to "%u:%u"
> >>    (see recent hackers thread)
> > 
> > !     printf(_("Latest checkpoint's NextXID:          %u/%u\n"),
> >              ControlFile.checkPointCopy.nextXidEpoch,
> >              ControlFile.checkPointCopy.nextXid);
> >       printf(_("Latest checkpoint's NextOID:          %u\n"),
> > --- 646,652 ----
> >              ControlFile.checkPointCopy.ThisTimeLineID);
> >       printf(_("Latest checkpoint's full_page_writes: %s\n"),
> >              ControlFile.checkPointCopy.fullPageWrites ? _("on") : _("off"));
> > !     printf(_("Latest checkpoint's NextXID:          %u:%u\n"),
> > This should be definitely a separate patch.
> 
> Ok. Notwithstanding Simon's reply, there seems to be consensus that this
> is the way to go. Will commit it this way unless some additional
> objections surface in the next day or so.

FYI, this slash-colon change will break pg_upgrade unless it is patched.
Dp you want a patch from me?

--  Bruce Momjian  <bruce@momjian.us>        http://momjian.us EnterpriseDB
http://enterprisedb.com

+ As you are, so once was I. As I am, so you will be. +
+ Roman grave inscription                             +