Re: GIN pageinspect support for entry tree and posting tree

From: Japin Li <japinli(at)hotmail(dot)com>
To: Andrey Borodin <x4mmm(at)yandex-team(dot)ru>
Cc: Kirill Reshke <reshkekirill(at)gmail(dot)com>, Roman Khapov <rkhapov(at)yandex-team(dot)ru>, Chao Li <li(dot)evan(dot)chao(at)gmail(dot)com>, "Peter Eisentraut" <peter(at)eisentraut(dot)org>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: GIN pageinspect support for entry tree and posting tree
Date: 2026-09-02 02:08:08
Message-ID: SY7PR01MB10921891275CC615D3282326EB6B72@SY7PR01MB10921.ausprd01.prod.outlook.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, 02 Sep 2026 at 00:10, Japin Li <japinli(at)hotmail(dot)com> wrote:
> Hi, Andrey and Kirill
>
> On Sun, 30 Aug 2026 at 22:46, Andrey Borodin <x4mmm(at)yandex-team(dot)ru> wrote:
>>> On 3 Aug 2026, at 17:40, Kirill Reshke <reshkekirill(at)gmail(dot)com> wrote:
>>>
>>> Hi all,
>>>
>>> I have rebased patches and did little polishing for error
>>> messages/overall style. Posting to see how new CI reacts & also bump
>>> thread.
>>>
>> Hi Kirill,
>>
>> The CF entry is still Ready for Committer, but Japin's August 13 review
>> found that gin_entrypage_items() does not finalize its temporary tuple
>> descriptor before index_getattr(). With assertions enabled, the new
>> check added by c456e391138 can fail here.
>>
>> Could you fold the proposed TupleDescFinalize() call into a new patch
>> version? I have moved the entry to Waiting on Author for now.
>>
>
> The v20260308 patches need a rebase due to commit 51c43a5dbd86. I've rebased
> and merged the fix from [1]. Please take a look.
>
> [1] https://www.postgresql.org/message-id/SY7PR01MB109210C8E9DEB6BF5FFEA7A81B6DB2%40SY7PR01MB10921.ausprd01.prod.outlook.com
>

Sorry, I forgot to update the patches.

--
Regards,
Japin Li
ChengDu WenWu Information Technology Co., Ltd.

Attachment Content-Type Size
v20260901-0001-Modernize-coding-in-GIN-pageinspect-functi.patch text/x-patch 5.3 KB
v20260901-0002-GIN-pageinspect-support-for-entry-tree-and.patch text/x-patch 31.9 KB
v20260901-0003-Move-IS_INDEX-macro-to-pageinspect.h.patch text/x-patch 4.0 KB

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Chao Li 2026-09-02 02:13:48 Re: Include sequences in publications created by pg_createsubscriber
Previous Message shihao zhong 2026-09-02 02:02:22 Re: [PATCH v4] Add pg_current_vxact_id() function