Gregory Stark <[EMAIL PROTECTED]> writes:
> "Tom Lane" <[EMAIL PROTECTED]> writes:
>> On investigation the problem occurs because we changed vacuum.c's
>> PageGetFreeSpaceWithFillFactor() to use PageGetHeapFreeSpace()
>> instead of just computing pd_upper - pd_lower as it had done in
>> every previous release.  This was *not* a good idea: VACUUM FULL
>> does its own accounting for line pointers and does not need "help".

> Fwiw this change appears to have crept in when the patch was merged.

Yeah, it's entirely likely that this was my fault :-(.  It wasn't
apparent at the time that this change wasn't safe and conservative...

                        regards, tom lane

---------------------------(end of broadcast)---------------------------
TIP 3: Have you checked our extensive FAQ?

               http://www.postgresql.org/docs/faq

Reply via email to