From: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com> |
---|---|
To: | Magnus Hagander <magnus(at)hagander(dot)net> |
Cc: | Cédric Villemain <cedric(dot)villemain(dot)debian(at)gmail(dot)com>, Dimitri Fontaine <dimitri(at)2ndquadrant(dot)fr>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Streaming base backups |
Date: | 2011-01-12 09:39:05 |
Message-ID: | AANLkTik4QQWe5-m5uro-rNzkT4CH8TXPPvOr_9bOQMz3@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Jan 10, 2011 at 11:09 PM, Magnus Hagander <magnus(at)hagander(dot)net> wrote:
> I've committed the backend side of this, without that. Still working
> on the client, and on cleaning up Heikki's patch for grammar/parser
> support.
Great work!
I have some comments:
While walsender is sending a base backup, WalSndWakeup should
not send the signal to that walsender?
In sendFile or elsewhere, we should periodically check whether
postmaster is alive and whether the flag was set by the signal?
At the end of the backup by walsender, it forces a switch to a new
WAL file and waits until the last WAL file has been archived. So we
should change postmaster so that it doesn't cause the archiver to
end before walsender ends when shutdown is requested?
Also, when shutdown is requested, the walsender which is
streaming WAL should not end before another walsender which
is sending a backup ends, to stream the backup-end WAL?
Regards,
--
Fujii Masao
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center
From | Date | Subject | |
---|---|---|---|
Next Message | Itagaki Takahiro | 2011-01-12 10:05:47 | Re: multiset patch review |
Previous Message | Dimitri Fontaine | 2011-01-12 09:36:39 | Re: Add function dependencies |