Re: exposing pg_controldata and pg_config as functions - Mailing list pgsql-hackers

On 01/18/2016 01:47 PM, Bruce Momjian wrote:
> 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?

Didn't realize that -- yes please.

Thanks,

Joe

--
Crunchy Data - http://crunchydata.com
PostgreSQL Support for Secure Enterprises
Consulting, Training, & Open Source Development