pgsql: Avoid including utils/timestamp.h in conflict.h. - Mailing list pgsql-committers

From Amit Kapila
Subject pgsql: Avoid including utils/timestamp.h in conflict.h.
Date
Msg-id E1vuNxs-000kZu-39@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Avoid including utils/timestamp.h in conflict.h.

conflict.h currently includes utils/timestamp.h despite only requiring
basic timestamp type definitions. This creates unnecessary overhead.

Replace the include with datatype/timestamp.h to provide the necessary
types. This change requires explicitly including utils/timestamp.h in
test_custom_fixed_stats.c, which previously relied on the indirect
inclusion.

Extracted from the larger patch by Andres Freund.
Discussion: https://postgr.es/m/aY-UE-4t7FiYgH3t@alap3.anarazel.de

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/308622edf1743aa407df0044cc4529aed8de15e6

Modified Files
--------------
src/include/replication/conflict.h                           | 2 +-
src/test/modules/test_custom_stats/test_custom_fixed_stats.c | 1 +
2 files changed, 2 insertions(+), 1 deletion(-)


pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: pgsql: doc: Add section "Options" for pg_controldata
Next
From: Peter Eisentraut
Date:
Subject: pgsql: Fix additional fallthrough warnings from clang