From: | "MauMau" <maumau307(at)gmail(dot)com> |
---|---|
To: | "Fujii Masao" <masao(dot)fujii(at)gmail(dot)com> |
Cc: | <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [9.3 bug] disk space in pg_xlog increases during archive recovery |
Date: | 2013-07-31 22:37:58 |
Message-ID: | D924375477D34942A6B421B5B82123E7@maumau |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
From: "Fujii Masao" <masao(dot)fujii(at)gmail(dot)com>
> - if (source == XLOG_FROM_ARCHIVE)
> + if (source == XLOG_FROM_ARCHIVE &&
> + StandbyModeRequested && AllowCascadeReplication())
>
> I think that the condition of StandbyModeRequested should be removed
> because someone might want to set up the cascade standby from the standby
> of warm-standby configuration.
Fixed and attached the revised patch.
However, isn't StandbyRequested true (= standby_mode set to on) to enable
warm standby? I'm afraid people set max_wal_senders>0 and hot_standby=on
even on the primary server to make the contents of postgresql.conf identical
on both the primary and the standby for easier configuration. If so, normal
archive recovery (PITR, not the standby recovery) would face the original
problem -- unnecessary WAL accumulation in pg_xlog/. So I'm wonder if
AllowCascadeReplication() is enough.
Please take either this patch or the previous one.
Regards
MauMau
Attachment | Content-Type | Size |
---|---|---|
skip_wal_save_v2.patch | application/octet-stream | 891 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | David Fetter | 2013-07-31 22:58:37 | Re: pass-through queries to foreign servers |
Previous Message | Alvaro Herrera | 2013-07-31 20:55:33 | Re: 9.3beta2: Failure to pg_upgrade |