From: | Simon Riggs <simon(at)2ndquadrant(dot)com> |
---|---|
To: | Fujii Masao <masao(dot)fujii(at)gmail(dot)com> |
Cc: | Noah Misch <noah(at)leadboat(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Prevent restored WAL files from being archived again Re: Unnecessary WAL archiving after failover |
Date: | 2012-08-09 14:24:39 |
Message-ID: | CA+U5nMLxKseoEG5zVFY_qVrUY7quq9_Fv+5atTiR-B2uxTdMqg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 9 August 2012 00:08, Simon Riggs <simon(at)2ndquadrant(dot)com> wrote:
> Will look at the refactored version tomorrow.
Rather than implement the minimal patch onto HEAD, I've written a
refactored patch. [attached]
This fixes the bugs discussed here, though allows for multiple archive
statuses, rather than just two, as well as removing similar code.
We've talked about allowing file based replication to cascade, i.e.
having the archiver run on standby and to push files onwards from the
standby.
The way I'd like to do this is
If archiving active
* Allow archiver to run on standby
* When xlog file written on standby it sets archive_status to ".standby"
* When archiver runs it will execute archive_command when it sees a
status of ".ready" or it will execute standby_archive_command when it
sees a status of ".standby". In both cases it will set a status of
".done" when complete. It doesn't need to know whether promotion has
happened, or even when it happened, because the status of the xlog
files is clearly marked.
We can't use ".ready" to mean two things because then we'd be confused
how to handle files generated during standby but archived after
promotion has taken place. ".done" always means you can clean up the
xlog files now, in all cases.
You're welcome to complete the implementation using this patch, or
I'll do that when I get back from leave.
--
Simon Riggs http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
Attachment | Content-Type | Size |
---|---|---|
refactor_archive_status.v1.patch | application/octet-stream | 10.4 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2012-08-09 14:27:12 | Re: patch submission: truncate trailing nulls from heap rows to reduce the size of the null bitmap |
Previous Message | Kevin Grittner | 2012-08-09 14:22:38 | Wiki link for max_connections? (Fwd: Re: PostgreSQL oom_adj postmaster process to -17) |