Re: "Garbled" postgres logs - Mailing list pgsql-hackers

From Tom Lane
Subject Re: "Garbled" postgres logs
Date
Msg-id 12421.1175633493@sss.pgh.pa.us
Whole thread Raw
In response to Re: "Garbled" postgres logs  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: "Garbled" postgres logs  (tgoodair@ca.afilias.info (Tim Goodaire))
List pgsql-hackers
Andrew Dunstan <andrew@dunslane.net> writes:
> Tom Lane wrote:
>> Note to hackers: would it make sense to use write() instead of
>> fprintf() in send_message_to_server_log to avoid any possibility
>> of stdio deciding to fragment the message?  Possibly there'd be
>> some marginal efficiency gain too.

> What about in write_syslogger_file_binary()? Since redirect_stderr is 
> set true in the case reported, wont that be what does the writing?

No, syslogger is single-threaded so it can't be at fault.  The
interleaving must be happening when the data is inserted into the pipe
that leads to syslogger.  We've got multiple backends concurrently
writing that pipe, remember.

BTW, although I'm blaming stdio here, it's conceivable that it is
issuing messages in a single write() and the kernel is failing to keep
the writes atomic, as I think is required by spec if the write is for
less than PIPEBUF bytes.  So Tim might want to compare exact kernel
versions as well as exact libc versions between the misbehaving machine
and the others.
        regards, tom lane


pgsql-hackers by date:

Previous
From: Brad Nicholson
Date:
Subject: Re: "Garbled" postgres logs
Next
From: "Luke Lonergan"
Date:
Subject: Re: Oracle indemnifies PostgreSQL on its patents