Reduce header inclusions via execnodes.h Remove a bunch of #include lines from execnodes.h. Most of these requier suitable typedefs to be added, so that it still compiles standalone. In one case, the fix is to move a struct definition to the one .c file where it is needed.
Also some light clean up in plannodes.h and genam.h, though not as extensive as in execnodes.h. Author: Álvaro Herrera <[email protected]> Author: Andres Freund <[email protected]> Discussion: https://postgr.es/m/[email protected] Branch ------ master Details ------- https://git.postgresql.org/pg/commitdiff/fba4233c832870c8363438419743c48fdcb2151c Modified Files -------------- contrib/amcheck/verify_heapam.c | 1 + contrib/dblink/dblink.c | 1 + contrib/hstore/hstore_subs.c | 1 + contrib/pageinspect/brinfuncs.c | 1 + contrib/pageinspect/gistfuncs.c | 1 + contrib/pg_buffercache/pg_buffercache_pages.c | 1 + contrib/pg_stat_statements/pg_stat_statements.c | 1 + contrib/pg_walinspect/pg_walinspect.c | 2 + contrib/pgrowlocks/pgrowlocks.c | 1 + contrib/postgres_fdw/connection.c | 1 + contrib/postgres_fdw/postgres_fdw.c | 1 + contrib/tablefunc/tablefunc.c | 1 + contrib/tsm_system_time/tsm_system_time.c | 1 + contrib/xml2/xpath.c | 1 + src/backend/access/brin/brin.c | 2 + src/backend/access/gin/gininsert.c | 3 + src/backend/access/heap/heapam_handler.c | 1 + src/backend/access/nbtree/nbtsort.c | 1 + src/backend/access/transam/xlogprefetcher.c | 2 + src/backend/backup/walsummaryfuncs.c | 1 + src/backend/bootstrap/bootstrap.c | 1 + src/backend/catalog/index.c | 1 + src/backend/commands/analyze.c | 1 + src/backend/commands/copyfrom.c | 1 + src/backend/commands/copyfromparse.c | 1 + src/backend/commands/copyto.c | 1 + src/backend/commands/event_trigger.c | 1 + src/backend/commands/explain.c | 1 + src/backend/commands/extension.c | 1 + src/backend/commands/indexcmds.c | 1 + src/backend/commands/prepare.c | 2 + src/backend/commands/tablecmds.c | 1 + src/backend/commands/trigger.c | 2 + src/backend/commands/wait.c | 1 + src/backend/executor/execAmi.c | 1 + src/backend/executor/execAsync.c | 1 + src/backend/executor/execExprInterp.c | 2 + src/backend/executor/execMain.c | 2 + src/backend/executor/execPartition.c | 1 + src/backend/executor/execProcnode.c | 1 + src/backend/executor/execSRF.c | 1 + src/backend/executor/execUtils.c | 2 + src/backend/executor/functions.c | 1 + src/backend/executor/nodeAgg.c | 2 + src/backend/executor/nodeBitmapAnd.c | 2 + src/backend/executor/nodeBitmapHeapscan.c | 40 ++++++++++++ src/backend/executor/nodeBitmapIndexscan.c | 2 + src/backend/executor/nodeBitmapOr.c | 2 + src/backend/executor/nodeCtescan.c | 1 + src/backend/executor/nodeFunctionscan.c | 1 + src/backend/executor/nodeGatherMerge.c | 1 + src/backend/executor/nodeGroup.c | 1 + src/backend/executor/nodeHash.c | 1 + src/backend/executor/nodeHashjoin.c | 1 + src/backend/executor/nodeIndexonlyscan.c | 1 + src/backend/executor/nodeIndexscan.c | 2 + src/backend/executor/nodeMaterial.c | 1 + src/backend/executor/nodeMergeAppend.c | 1 + src/backend/executor/nodeMergejoin.c | 2 + src/backend/executor/nodeModifyTable.c | 2 + src/backend/executor/nodeNamedtuplestorescan.c | 1 + src/backend/executor/nodeNestloop.c | 1 + src/backend/executor/nodeRecursiveunion.c | 1 + src/backend/executor/nodeSetOp.c | 1 + src/backend/executor/nodeTableFuncscan.c | 1 + src/backend/executor/nodeWindowAgg.c | 2 + src/backend/executor/nodeWorktablescan.c | 1 + src/backend/executor/spi.c | 1 + src/backend/foreign/foreign.c | 1 + src/backend/jit/jit.c | 1 + src/backend/optimizer/path/costsize.c | 1 + src/backend/optimizer/plan/initsplan.c | 1 + src/backend/optimizer/prep/preptlist.c | 2 + src/backend/optimizer/prep/prepunion.c | 1 + src/backend/optimizer/util/appendinfo.c | 1 + src/backend/optimizer/util/predtest.c | 1 + src/backend/parser/parse_utilcmd.c | 1 + src/backend/postmaster/autovacuum.c | 1 + src/backend/replication/logical/worker.c | 2 + src/backend/rewrite/rewriteGraphTable.c | 1 + src/backend/rewrite/rewriteManip.c | 1 + src/backend/storage/ipc/dsm_registry.c | 1 + src/backend/storage/ipc/shmem.c | 3 + src/backend/tcop/postgres.c | 1 + src/backend/utils/activity/wait_event_funcs.c | 1 + src/backend/utils/adt/acl.c | 1 + src/backend/utils/adt/arraysubs.c | 1 + src/backend/utils/adt/datetime.c | 1 + src/backend/utils/adt/hbafuncs.c | 1 + src/backend/utils/adt/json.c | 1 + src/backend/utils/adt/jsonbsubs.c | 1 + src/backend/utils/adt/jsonfuncs.c | 1 + src/backend/utils/adt/mcxtfuncs.c | 2 + src/backend/utils/adt/misc.c | 1 + src/backend/utils/adt/pgstatfuncs.c | 1 + src/backend/utils/adt/rangetypes.c | 2 + src/backend/utils/adt/varlena.c | 2 + src/backend/utils/cache/relcache.c | 1 + src/backend/utils/fmgr/fmgr.c | 1 + src/backend/utils/init/postinit.c | 1 + src/backend/utils/misc/guc_funcs.c | 2 + src/backend/utils/misc/pg_config.c | 1 + src/backend/utils/mmgr/portalmem.c | 2 + src/backend/utils/sort/tuplestore.c | 1 + src/include/access/genam.h | 10 +-- src/include/access/gin_private.h | 1 + src/include/bootstrap/bootstrap.h | 1 + src/include/catalog/index.h | 6 ++ src/include/commands/explain.h | 1 + src/include/commands/explain_state.h | 1 + src/include/executor/execScan.h | 1 + src/include/executor/executor.h | 1 + src/include/executor/hashjoin.h | 2 + src/include/lib/simplehash.h | 3 +- src/include/nodes/execnodes.h | 76 ++++++++-------------- src/include/nodes/plannodes.h | 2 - src/pl/plperl/plperl.c | 1 + src/pl/tcl/pltcl.c | 1 + .../modules/injection_points/injection_points.c | 1 + .../test_custom_stats/test_custom_var_stats.c | 1 + .../modules/test_ddl_deparse/test_ddl_deparse.c | 1 + src/test/modules/test_regex/test_regex.c | 1 + 122 files changed, 225 insertions(+), 56 deletions(-)
