| 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 |
| 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() |