Re: test_compression, test module for low-level compression APIs (for 2b5ba2a0a141)

Michael Paquier <michael@paquier.xyz>

From: Michael Paquier <michael@paquier.xyz>
To: Andres Freund <andres@anarazel.de>
Cc: Postgres hackers <pgsql-hackers@lists.postgresql.org>, Andrew Dunstan <andrew@dunslane.net>
Date: 2026-04-13T19:01:53Z
Lists: pgsql-hackers
On Mon, Apr 13, 2026 at 02:50:31PM -0400, Andres Freund wrote:
> Hi,
> 
> On 2026-04-14 02:13:52 +0900, Michael Paquier wrote:
> > +++ b/src/test/regress/parallel_schedule
>> @@ -128,7 +128,7 @@ test: partition_merge partition_split partition_join partition_prune reloptions
>>  # event_trigger depends on create_am and cannot run concurrently with
>>  # any test that runs DDL
>>  # oidjoins is read-only, though, and should run late for best coverage
>> -test: oidjoins event_trigger
>> +test: oidjoins event_trigger compression_pglz
> 
> The new test creates a function, so I don't think this will be safe?

Yep, that was a thinko.  The group for plancache still has room for
one test, as it can go up to 19, so we could move one test in the
group of partition_merge t the group of plancache, keeping all the
compression tests together.

Anyway, how about just creating a new group for all the compression
tests?  I have a patch set in preparation for zstd and TOAST, where I
would put the new test suite in this group.
--
Michael

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →
  1. Add tests for low-level PGLZ [de]compression routines

  2. Fix heap-buffer-overflow in pglz_decompress() on corrupt input.