From: | Simon Riggs <simon(at)2ndquadrant(dot)com> |
---|---|
To: | Heikki Linnakangas <heikki(at)enterprisedb(dot)com> |
Cc: | Kevin Grittner <Kevin(dot)Grittner(at)wicourts(dot)gov>, Jeff Davis <pgsql(at)j-davis(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [FEATURE REQUEST] Streaming Onlinebackup (MaybeOFFTOPIC) |
Date: | 2007-09-27 08:01:28 |
Message-ID: | 1190880088.4194.17.camel@ebony.site |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, 2007-09-27 at 08:31 +0100, Heikki Linnakangas wrote:
> Kevin Grittner wrote:
> > <1190837675(dot)4181(dot)716(dot)camel(at)ebony(dot)site>, Simon Riggs <simon(at)2ndquadrant(dot)com>
> > wrote:
> >> We should also document that this is designed to help compress files
> >> that aren't full because we switched early because of archive_timeout.
> >
> > Attached is a modified version to implement both of these. I also bailed
> > out if there was surplus input. I tried an optimization of allocating a
> > separate buffer for outputting the zeros, to avoid repeated memset calls.
> > It didn't seem to make a very big difference; do you think it's worth
> > cluttering the code with that?
>
> Would it work to just ftruncate the file?
At the source end, yes. At the destination one of the tests we perform
for a fully and correctly copied file is the file size; a smaller file
size we take to mean that the copy is still in progress. We check that
before we actually read the file, so it would complicate things
considerably to allow for variable file sizes.
--
Simon Riggs
2ndQuadrant http://www.2ndQuadrant.com
From | Date | Subject | |
---|---|---|---|
Next Message | Zeugswetter Andreas ADI SD | 2007-09-27 08:17:06 | Re: [FEATURE REQUEST] Streaming Onlinebackup (MaybeOFFTOPIC) |
Previous Message | Heikki Linnakangas | 2007-09-27 07:31:20 | Re: [FEATURE REQUEST] Streaming Onlinebackup (MaybeOFFTOPIC) |