Re: pgsql: Add -f/--follow option to pg_xlogdump. - Mailing list pgsql-committers

From Heikki Linnakangas
Subject Re: pgsql: Add -f/--follow option to pg_xlogdump.
Date
Msg-id 5332D5DF.9060408@vmware.com
Whole thread Raw
In response to Re: pgsql: Add -f/--follow option to pg_xlogdump.  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: pgsql: Add -f/--follow option to pg_xlogdump.  (Andrew Dunstan <andrew@dunslane.net>)
List pgsql-committers
On 03/26/2014 03:15 PM, Andrew Dunstan wrote:
>
> On 03/26/2014 07:50 AM, Heikki Linnakangas wrote:
>> Add -f/--follow option to pg_xlogdump.
>>
>> This is useful for seeing what WAL records are inserted in real-time, by
>> pointing pg_xlogdump to a live server.
>>
>> Branch
>> ------
>> master
>>
>> Details
>> -------
>> http://git.postgresql.org/pg/commitdiff/ce9bb92f8fb8d25cf00ec939797ffdb5930fb792
>>
>> Modified Files
>> --------------
>> contrib/pg_xlogdump/pg_xlogdump.c |   31 ++++++++++++++++++++++++++++---
>> doc/src/sgml/pg_xlogdump.sgml     |   11 +++++++++++
>> 2 files changed, 39 insertions(+), 3 deletions(-)
>>
>
>
> Seems to have upset Windows (see
> <http://www.pgbuildfarm.org/cgi-bin/show_log.pl?nm=currawong&dt=2014-03-26%2012%3A30%3A44>).
>
> Shouldn't you be using pg_usleep() instead of sleep()?

Apparently. Fixed that way.

I'm surprised that pg_upgrade doesn't have the same problem - there's a
plain sleep(1) call in contrib/pg_upgrade/exec.c. What's the difference?

- Heikki


pgsql-committers by date:

Previous
From: Heikki Linnakangas
Date:
Subject: pgsql: Use pg_usleep() instead of plain sleep(), to fix Windows build
Next
From: Andrew Dunstan
Date:
Subject: Re: pgsql: Add -f/--follow option to pg_xlogdump.