Hi,
On Wed, Jul 15, 2009 at 3:56 AM, Heikki
Linnakangas<[email protected]> wrote:
> Here's one little thing in addition to all the stuff already discussed:
Thanks for the comment!
> If that's the only such caller, let's introduce a new function for that
> and keep the XLogFlush() api unchanged.
OK. How about the following function?
------------------
/*
* Ensure that shutdown-related XLOG data through the given position is
* flushed to local disk, and also flushed to the disk in the standby
* if replication is in progress.
*/
void
XLogShutdownFlush(XLogRecPtr record)
{
int save_mode = XLogSyncReplication;
XLogSyncReplication = REPLICATION_MODE_FSYNC;
XLogFlush(record);
XLogSyncReplication = save_mode;
}
------------------
In a shutdown checkpoint case, CreateCheckPoint calls
XLogShutdownFlush, otherwise XLogFlush. And,
XLogFlush uses XLogSyncReplication directly instead of
obsolete 'mode' argument.
If the above is OK, should I update the patch ASAP? or
suspend that update until many other comments arrive?
I'm concerned that frequent small updating interferes with
a review.
Regards,
--
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center
--
Sent via pgsql-hackers mailing list ([email protected])
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers
