From: | Koichi Suzuki <suzuki(dot)koichi(at)oss(dot)ntt(dot)co(dot)jp> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Josh Berkus <josh(at)agliodbs(dot)com>, Zeugswetter Andreas ADI SD <ZeugswetterA(at)spardat(dot)at>, pgsql-hackers(at)postgresql(dot)org, pgsql-patches(at)postgresql(dot)org |
Subject: | Re: [PATCHES] Full page writes improvement, code update |
Date: | 2007-05-07 12:44:34 |
Message-ID: | 463F1F32.5070707@oss.ntt.co.jp |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers pgsql-patches |
Hi,
As replied to "Patch queue triage" by Tom, here's simplified patch to
mark WAL record as "compressable", with no increase in WAL itself.
Compression/decompression commands will be posted separately to PG
Foundary for further review.
-------------------------------
As suggested by Tom, I agree that WAL should not include "both" full
page write and incremental (logical) log. I began to examine WAL
record format to see if incremental log can be made from full page
writes. It will be okay even before 8.4, if simplified patch to the
core is accepted. I will post simplified patch to the core as follows:
1. Mark the flag to indicate that the WAL record is compressable from
full page writes to incremental log. This flag will be set if
a) It is not written during the hot backup, and
b) Archive command is active, and
c) WAL contains full page writes, and
d) full_page_writes=on.
No logical log will be written to WAL in this case, and
2. During recovery, xl_tot_len check will be skipped for compressed WAL
records.
Please note that new GUC is not needed in this patch.
With this patch, compress/decompress can be developped outside the core.
I'd be very grateful if this patch can be considered again.
Best Regards;
--
-------------
Koichi Suzuki
Attachment | Content-Type | Size |
---|---|---|
lesslog_core.patch_flagonly | text/plain | 3.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Zdenek Kotala | 2007-05-07 12:47:14 | Re: New idea for patch tracking |
Previous Message | Jim Nasby | 2007-05-07 12:36:55 | Re: Managing the community information stream |
From | Date | Subject | |
---|---|---|---|
Next Message | Peter Eisentraut | 2007-05-07 15:16:28 | Re: Diagnostic functions |
Previous Message | Pavan Deolasee | 2007-05-07 12:03:33 | HOT patches |