On Tuesday, May 31, 2011 02:14:00 AM Andres Freund wrote:
> On Tuesday, May 31, 2011 01:56:05 AM C=E9dric Villemain wrote:
> > I remove my own explanations as we conclude on the same thing.
> > Attached is the fix by adding a (!reindex) in the index.c if().
>=20
> Thats imo wrong because it will break a plain REINDEX?
>=20
> I think one possible correct fix would be the attached:
My version was wrong as well because it did not observe RelationTruncate's=
=20
nblocks argument. That function is used to "shorten" the relation in vacuum=
.=20
So dropping the init fork there is not a good idea.
So I think it is the simpler version of simply checking the existance of th=
e=20
fork before creating is ok.
Andres