Re: Re: [COMMITTERS] pgsql: Checkpointer starts before bgwriter to avoid missing fsync reque

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Simon Riggs <simon(at)2ndQuadrant(dot)com>
Cc: Heikki Linnakangas <heikki(dot)linnakangas(at)enterprisedb(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Re: [COMMITTERS] pgsql: Checkpointer starts before bgwriter to avoid missing fsync reque
Date: 2012-06-01 13:59:23
Message-ID: 24176.1338559163@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers pgsql-hackers

Simon Riggs <simon(at)2ndQuadrant(dot)com> writes:
> On 1 June 2012 14:29, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Surely that commit is useless. Fsync requests go into a queue in shared
>> memory, which had better have been set up by the postmaster. There is
>> no requirement that the receiving process exist before somebody can put
>> a request into the queue. If the queue overflows, the requestor has to
>> take care of the fsync itself, but that is independent of whether the
>> checkpointer is running yet.

> The problem I saw was about fsync queue message overflow, not actually
> missing fsyncs, so perhaps I worded the commit message poorly.

Ah. Well, as long as the overflowed fsyncs do get handled on the
requesting side, I see no bug here. No objection to changing the order
in which we launch the processes, but as Heikki says, it's not clear
that that is really going to make much difference.

regards, tom lane

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Simon Riggs 2012-06-01 14:30:28 Re: Re: [COMMITTERS] pgsql: Checkpointer starts before bgwriter to avoid missing fsync reque
Previous Message Simon Riggs 2012-06-01 13:49:39 Re: Re: [COMMITTERS] pgsql: Checkpointer starts before bgwriter to avoid missing fsync reque

Browse pgsql-hackers by date

  From Date Subject
Next Message Simon Riggs 2012-06-01 14:30:28 Re: Re: [COMMITTERS] pgsql: Checkpointer starts before bgwriter to avoid missing fsync reque
Previous Message Tom Lane 2012-06-01 13:52:59 Re: pg_upgrade from 9.0.7 to 9.1.3: duplicate key pg_authid_oid_index