On Wed, Jun 21, 2023 at 11:53 AM Peter Geoghegan <p...@bowt.ie> wrote: > > On Tue, Jun 20, 2023 at 10:27 PM Andres Freund <and...@anarazel.de> wrote: > > As far as I can tell 72e78d831a as-is is just bogus. Unfortunately that > > likely > > also means 3ba59ccc89 is not right. > > Quite possibly. But I maintain that ginInsertCleanup() is probably > also bogus in a way that's directly relevant. > > Did you know that ginInsertCleanup() is the only code that uses > heavyweight page locks these days? Though only on the index metapage! > > Isn't this the kind of thing that VACUUM's relation level lock is > supposed to take care of? >
Yeah, I also can't see why that shouldn't be sufficient for VACUUM. Assuming your observation is correct, what do you suggest doing in this regard? -- With Regards, Amit Kapila.