Re: Pass heaprel to GlobalVisTestFor() in vacuumRedirectAndPlaceholder()

From: Peter Geoghegan <pg(at)bowt(dot)ie>
To: "Drouvot, Bertrand" <bertranddrouvot(dot)pg(at)gmail(dot)com>
Cc: PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: Pass heaprel to GlobalVisTestFor() in vacuumRedirectAndPlaceholder()
Date: 2023-04-02 17:22:18
Message-ID: CAH2-Wz=eKLCO6vafTtM6QT=CSWAxA51ZtJors6MjfZukyXmRUg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sun, Apr 2, 2023 at 10:18 AM Peter Geoghegan <pg(at)bowt(dot)ie> wrote:
> Making nbtree page deletion more efficient when logical decoding is in
> use seems well worthwhile. There is an "XXX" comment about this issue,
> similar to the SP-GiST one. It looks like you already have everything
> you need to make this work from yesterday's commit 61b313e47e.

Actually, I suppose that isn't quite true, since you'd still need to
find a way to pass the heap relation down to nbtree VACUUM. Say by
adding it to IndexVacuumInfo.

That doesn't seem hard at all. The hard part was passing the heap rel
down to _bt_getbuf(), which you've already taken care of.

--
Peter Geoghegan

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Daniel Gustafsson 2023-04-02 17:37:55 Re: Add "host" to startup packet
Previous Message Peter Geoghegan 2023-04-02 17:18:10 Re: Pass heaprel to GlobalVisTestFor() in vacuumRedirectAndPlaceholder()