diff options
| author | Tom Lane | 2019-12-11 22:05:30 +0000 |
|---|---|---|
| committer | Tom Lane | 2019-12-11 22:05:30 +0000 |
| commit | 5935917ce59e2e613ac7a4b54ed49a7b9f8f28ac (patch) | |
| tree | b89e6f02a9c226b1e6e94ddf8b3de73afd64a761 /src/include/nodes | |
| parent | 6ef77cf46e81f45716ec981cb08781d426181378 (diff) | |
Allow executor startup pruning to prune all child nodes.
Previously, if the startup pruning logic proved that all child nodes
of an Append or MergeAppend could be pruned, we still kept one, just
to keep EXPLAIN from failing. The previous commit removed the
ruleutils.c limitation that required this kluge, so drop it. That
results in less-confusing EXPLAIN output, as per a complaint from
Yuzuko Hosoya.
David Rowley
Discussion: https://postgr.es/m/001001d4f44b$2a2cca50$7e865ef0$@lab.ntt.co.jp
Diffstat (limited to 'src/include/nodes')
| -rw-r--r-- | src/include/nodes/execnodes.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/src/include/nodes/execnodes.h b/src/include/nodes/execnodes.h index ea93b769e1..0c2a77aaf8 100644 --- a/src/include/nodes/execnodes.h +++ b/src/include/nodes/execnodes.h @@ -1234,8 +1234,6 @@ struct AppendState * slots current output tuple of each subplan * heap heap of active tuples * initialized true if we have fetched first tuple from each subplan - * noopscan true if partition pruning proved that none of the - * mergeplans can contain a record to satisfy this query. * prune_state details required to allow partitions to be * eliminated from the scan, or NULL if not possible. * valid_subplans for runtime pruning, valid mergeplans indexes to @@ -1252,7 +1250,6 @@ typedef struct MergeAppendState TupleTableSlot **ms_slots; /* array of length ms_nplans */ struct binaryheap *ms_heap; /* binary heap of slot indices */ bool ms_initialized; /* are subplans started? */ - bool ms_noopscan; struct PartitionPruneState *ms_prune_state; Bitmapset *ms_valid_subplans; } MergeAppendState; |
