Re: BUG #18960: Mistake in test test_simple_pipeline (libpq_pipeline.c)

Tom Lane <tgl@sss.pgh.pa.us>

From: Tom Lane <tgl@sss.pgh.pa.us>

To: Dmitry Kovalenko <d.kovalenko@postgrespro.ru>

Cc: Álvaro Herrera <alvherre@kurilemu.de>, pgsql-bugs@lists.postgresql.org

Date: 2025-06-21T21:46:18Z

Lists: pgsql-bugs

Commits

Same data as JSON: GET /api/v1/messages/:b64id/commits the thread's linked commits as JSON, with link sources. API reference →

Make libpq_pipeline.c shorter and more uniform via helper functions.
- e351e5c4fea4 19 (unreleased) landed
pg_dump: Allow pg_dump to dump the statistics for foreign tables.
- c2e2589ab969 18.0 cited

Dmitry Kovalenko <d.kovalenko@postgrespro.ru> writes:
> As I understand, you have leaks of 'res' when (PQresultStatus(res) != 
> status)

I don't think we really care about leaks when pg_fatal is called.
(If we do, there are a lot of others to worry about.)

> I spent the some time to fix a leaks in tests and PG itself here:
> https://github.com/dmitry-lipetsk/postgres/commits/D20250617_001-pg_master/

Ah, I did not know you were working on this.  Perhaps you can merge
what we've done.

			regards, tom lane