Received: from maia.hub.org (maia-5.hub.org [200.46.204.29]) by mail.postgresql.org (Postfix) with ESMTP id 96622B5D95B for ; Thu, 27 Oct 2011 00:36:40 -0300 (ADT) Received: from mail.postgresql.org ([200.46.204.86]) by maia.hub.org (mx1.hub.org [200.46.204.29]) (amavisd-maia, port 10024) with ESMTP id 40483-05 for ; Thu, 27 Oct 2011 03:36:34 +0000 (UTC) X-Greylist: from auto-whitelisted by SQLgrey-1.8.0-rc2 Received: from sss.pgh.pa.us (sss.pgh.pa.us [66.207.139.130]) by mail.postgresql.org (Postfix) with ESMTP id 76681B5D80B for ; Thu, 27 Oct 2011 00:36:34 -0300 (ADT) Received: from sss2.sss.pgh.pa.us (tgl@localhost [127.0.0.1]) by sss.pgh.pa.us (8.14.2/8.14.2) with ESMTP id p9R3aJkw018334; Wed, 26 Oct 2011 23:36:19 -0400 (EDT) To: Robert Haas cc: Simon Riggs , Florian Pflug , Aidan Van Dyk , Daniel Farina , Chris Redekop , PostgreSQL-development Subject: Re: Hot Backup with rsync fails at pg_clog if under load In-reply-to: References: <213AEFBF-479C-4F9E-B0CA-24AAB2C9F4DD@phlo.org> <1A94CE1A-C3E5-4923-B677-623D41BC273F@phlo.org> <6C3D7EDA-573E-46DC-9047-5FEB92876DA8@phlo.org> <12E895D5-C6D0-4E71-B9C3-85E1BC14E6B8@phlo.org> Comments: In-reply-to Robert Haas message dated "Wed, 26 Oct 2011 21:32:31 -0400" Date: Wed, 26 Oct 2011 23:36:19 -0400 Message-ID: <18333.1319686579@sss.pgh.pa.us> From: Tom Lane X-Virus-Scanned: Maia Mailguard 1.0.1 X-Spam-Status: No, hits=-2.404 tagged_above=-5 required=5 tests=BAYES_00=-1.9, RP_MATCHES_RCVD=-0.504 X-Spam-Level: X-Archive-Number: 201110/1425 X-Sequence-Number: 197231 Robert Haas writes: > On Wed, Oct 26, 2011 at 12:16 PM, Simon Riggs wrote: >> This fixes both the subtrans and clog bugs in one patch. > I don't see the point of changing StartupCLOG() to be an empty > function and adding a new function TrimCLOG() that does everything > StartupCLOG() used to do. +1 ... I found that overly cute also. regards, tom lane