From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | Gregory Stark <stark(at)enterprisedb(dot)com> |
Cc: | "Zubkovsky\, Sergey" <Sergey(dot)Zubkovsky(at)transas(dot)com>, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: [DOCS] pg_total_relation_size() and CHECKPOINT |
Date: | 2008-03-14 20:02:01 |
Message-ID: | 24576.1205524921@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs pgsql-hackers |
Gregory Stark <stark(at)enterprisedb(dot)com> writes:
> "Tom Lane" <tgl(at)sss(dot)pgh(dot)pa(dot)us> writes:
>> Hmm. I find the whole thing fairly worrisome, because what it suggests
>> is that Windows isn't actually allocating file space during smgrextend,
>> which would mean that we'd be prone to running out of disk space at
>> unfortunate times --- like during a checkpoint, after we've already
>> promised the client the data is committed.
> Surely we can't lose after the fsync? Losing at commit rather than at
> the time of insert might still be poor, but how could we lose after
> we've promised the data is committed?
What I'm afraid of is write() returning ENOSPC for a write to a disk
block we thought we had allocated previously. If such a situation is
persistent we'd be unable to flush dirty data from shared buffers and
thus never be able to complete a checkpoint. We'd never *get* to the
fsync, so whether the data is safe after fsync is moot.
The way it is supposed to work is that ENOSPC ought to happen during
smgrextend, that is before we've put any data into a shared buffer
corresponding to a new page of the file. With that, we will never be
able to commit a transaction that requires disk space we don't have.
The real question here is whether Windows' stat() is telling the truth
about how much filesystem space has actually been allocated to a file.
It seems entirely possible that it's not; but if it is, then I think we
have a problem.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Knut P. Lehre | 2008-03-20 12:42:15 | Compiler settings, C-language functions for windows port |
Previous Message | Gregory Stark | 2008-03-14 18:30:24 | Re: [DOCS] pg_total_relation_size() and CHECKPOINT |
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2008-03-14 20:05:35 | Re: Commit fest? |
Previous Message | Alvaro Herrera | 2008-03-14 18:34:34 | Re: [COMMITTERS] pgsql: Fix vacuum so that autovacuum is really not cancelled when doing |