From: Álvaro Herrera <alvherre@kurilemu.de>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Don't include execnodes.h in replication/conflict.h
Date: Thu, 25 Sep 2025 12:53:52 +0000
Message-ID: <E1v1lTs-0000in-0m@gemulon.postgresql.org> (raw)
Don't include execnodes.h in replication/conflict.h
... which silently propagates a lot of headers into many places
via pgstat.h, as evidenced by the variety of headers that this patch
needs to add to seemingly random places. Add a minimum of typedefs to
conflict.h to be able to remove execnodes.h, and fix the fallout.
Backpatch to 18, where conflict.h first appeared.
Discussion: https://postgr.es/m/202509191927.uj2ijwmho7nv@alvherre.pgsql
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/7e638d7f5093fd24837ebe709135fa16ef8e3e7b
Modified Files
--------------
src/backend/access/transam/multixact.c | 1 +
src/backend/access/transam/xlogrecovery.c | 1 +
src/backend/storage/ipc/waiteventset.c | 1 +
src/backend/utils/activity/pgstat_backend.c | 1 +
src/include/pgstat.h | 1 +
src/include/replication/conflict.h | 9 ++++++++-
6 files changed, 13 insertions(+), 1 deletion(-)
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Reply to all the recipients using the --to and --cc options:
reply via email
To: pgsql-committers@postgresql.org
Cc: alvherre@kurilemu.de, pgsql-committers@lists.postgresql.org
Subject: Re: pgsql: Don't include execnodes.h in replication/conflict.h
In-Reply-To: <E1v1lTs-0000in-0m@gemulon.postgresql.org>
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox