Re: GIN pageinspect support for entry tree and posting tree - Mailing list pgsql-hackers

From Kirill Reshke
Subject Re: GIN pageinspect support for entry tree and posting tree
Date
Msg-id CALdSSPgZRJNr5RXswQqJw6n1vz_10psU5mnYmGfvWyPGexj9ww@mail.gmail.com
Whole thread Raw
In response to Re: GIN pageinspect support for entry tree and posting tree  (Andrey Borodin <x4mmm@yandex-team.ru>)
Responses Re: GIN pageinspect support for entry tree and posting tree
List pgsql-hackers
On Sat, 3 Jan 2026 at 23:58, Andrey Borodin <x4mmm@yandex-team.ru> wrote:
>
>
>
> > On 3 Jan 2026, at 00:16, Kirill Reshke <reshkekirill@gmail.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

pgsql-hackers by date:

Previous
From: Robert Haas
Date:
Subject: Re: confusing results from pg_get_replication_slots()
Next
From: Kirill Reshke
Date:
Subject: Re: [PATCH] psql: add size-based sorting options (O/o) for tables and indexes