Re: [PATCH] Compressed TOAST data corruption with REPACK CONCURRENTLY

From: Antonin Houska <ah(at)cybertec(dot)at>
To: alvherre(at)kurilemu(dot)de
Cc: "Michael Paquier" <michael(at)paquier(dot)xyz>, "SATYANARAYANA NARLAPURAM" <satyanarlapuram(at)gmail(dot)com>, "L(dot) pgsql-hackers" <pgsql-hackers(at)lists(dot)postgresql(dot)org>
Subject: Re: [PATCH] Compressed TOAST data corruption with REPACK CONCURRENTLY
Date: 2026-04-26 07:12:36
Message-ID: 12262.1777187556@localhost
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Antonin Houska <ah(at)cybertec(dot)at> wrote:

> Álvaro Herrera <alvherre(at)kurilemu(dot)de> wrote:
>
> > > ok, this is a patch to enhance the tests. It also simplifies
> > > gen_compressible() a bit and adjusts it so it is (hopefully) more reliable in
> > > generating compressible values. Now it can be used to reproduce the fix [1].
> > >
> > > [1] https://www.postgresql.org/message-id/52301.1776440752%40localhost
> >
> > Thanks, I think it looks good, but unfortunately I don't see any problems
> > when running this test without your fix (I was assuming that the test would
> > fail). I didn't immediately understand why. Do you see any failures?
>
> The enhanced tests should reproduce it (please let me know if they dont):
>
> https://www.postgresql.org/message-id/44015.1776685220%40localhost

I see you actually refer to the correct tests. I see no failure now as
well. I'll try to make the tests more stable. Sorry for the confusion.

--
Antonin Houska
Web: https://www.cybertec-postgresql.com

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Chao Li 2026-04-26 07:21:14 Re: Property graph: fix error handling when dropping non-existent label property
Previous Message Greg Burd 2026-04-25 20:08:02 [PATCH] Batched clock sweep to reduce cross-socket atomic contention