Stop including {brin,gin}_tuple.h in tuplesort.h
Doing this meant that those two headers, which are supposed to be
internal to their corresponding index AMs, were being included pretty
much universally, because tuplesort.h is included by execnodes.h which
is very widely used. Stop that, and fix fallout.
We also change indexing.h to no longer include execnodes.h (tuptable.h
is sufficient), and relscan.h to no longer include buf.h (pointless
since c2fe139c201c).
Author: Mario González <[email protected]>
Discussion:
https://postgr.es/m/CAFsReFUcBFup=Ohv_xd7SNQ=e73txi8ynektsfee2bw7js1...@mail.gmail.com
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/225d1df1d299061498182714d207fae77436702c
Modified Files
--------------
contrib/pageinspect/gistfuncs.c | 1 +
src/backend/access/gin/gininsert.c | 3 ++-
src/backend/catalog/pg_attrdef.c | 1 +
src/backend/catalog/pg_largeobject.c | 1 +
src/backend/commands/collationcmds.c | 1 +
src/backend/commands/user.c | 1 +
src/backend/executor/execReplication.c | 1 +
src/backend/parser/parse_expr.c | 1 +
src/backend/replication/logical/relation.c | 1 +
src/backend/replication/logical/sequencesync.c | 1 +
src/backend/statistics/stat_utils.c | 1 +
src/include/access/relscan.h | 1 -
src/include/catalog/indexing.h | 2 +-
src/include/nodes/execnodes.h | 2 ++
src/include/utils/tuplesort.h | 5 +++--
15 files changed, 18 insertions(+), 5 deletions(-)