Re: GIN pageinspect support for entry tree and posting tree

From: Kirill Reshke <reshkekirill(at)gmail(dot)com>
To: Andrey Borodin <x4mmm(at)yandex-team(dot)ru>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Roman Khapov <rkhapov(at)yandex-team(dot)ru>
Subject: Re: GIN pageinspect support for entry tree and posting tree
Date: 2026-01-03 19:25:49
Message-ID: CALdSSPgZRJNr5RXswQqJw6n1vz_10psU5mnYmGfvWyPGexj9ww@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Sat, 3 Jan 2026 at 23:58, Andrey Borodin <x4mmm(at)yandex-team(dot)ru> wrote:
>
>
>
> > On 3 Jan 2026, at 00:16, Kirill Reshke <reshkekirill(at)gmail(dot)com> wrote:
> >
> > PFA v4 with typo fixes.
>
> Thanks!
>
> I think block size dependency is not a problem. There are no pageinspect tests that pass with 1Kb pages.
>
> I have some more nits:
>
> Meson build needs to be updated.
>
> Documentation is on a TODO list.

Done

> + relation_close(indexRel, AccessShareLock);
> Perhaps, index_close() is what you actually wanted?

Ok

> StringInfoData buf is leaked still, you can init it once and reset in the loop.

Ok

> tmpTupdesc recreation worth commenting on.

Ok

> Thanks!
>
>
> Best regards, Andrey Borodin.

PFA v6

--
Best regards,
Kirill Reshke

Attachment Content-Type Size
v6-0001-GIN-pageinspect-support-for-entry-tree-and-postin.patch application/octet-stream 23.5 KB

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Kirill Reshke 2026-01-03 19:33:56 Re: [PATCH] psql: add size-based sorting options (O/o) for tables and indexes
Previous Message Robert Haas 2026-01-03 19:25:41 Re: confusing results from pg_get_replication_slots()