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