Re: New FSM patch

Tom Lane wrote:
> I did a bit of testing and immediately got an Assert failure:
> 
> ...
> 
> The scary part of that is that it gets through the regression tests ---
> doesn't leave one with a warm feeling about how much of VACUUM gets
> exercised by regression.

Ouch..

> I take it the comment at the top of indexfsm.c about using one bit per
> page should be recast as a possible future improvement?

Yep. I also noted that the code in GetIndexFreeSpace() didn't match the 
comment above it.

--   Heikki Linnakangas  EnterpriseDB   http://www.enterprisedb.com